Project

General

Profile

Refactor #29039

Make foreman::config::apache a standalone class

Added by Ewoud Kohl van Wijngaarden 8 months ago. Updated 8 months ago.


Description

By making the class standalone, it is easier to use it in other parts of the installer. It's also easier to test.

Associated revisions

Revision 8af63b84 (diff)
Added by Ewoud Kohl van Wijngaarden 8 months ago

Fixes #29039 - Make foreman::config::apache standalone

By not relying on the foreman class parameters, it becomes easier to use
this class without Foreman. This can be useful when deploying it using a
git checkout rather than packages.

Revision 386a1acc (diff)
Added by Ewoud Kohl van Wijngaarden 8 months ago

Refs #29039 - Use loose coupling to Foreman's Apache config

By only including the Apache config, we get only what we really need.

Revision f6dda3ec (diff)
Added by Ewoud Kohl van Wijngaarden 8 months ago

Refs #29039 - Use foreman's Apache config standalone (#324)

  • Refs #29039 - Use foreman's Apache config standalone

By using only the Apache config, it's possible to use the Apache config
without Foreman installed.

  • Move the repo_export_dir to application.pp

The repo export directory default matches with the
pulp_export_destination setting. This means it's only needed on a
Katello application server, not on a Pulp server.

By creating an anchor, we can collect this if needed and otherwise fall
back to a simple mkdir -p. By ordering these it should be idempotent,
guarantee the correct permissions on /var/lib/pulp and the
katello-export subdirectory.

History

#1 Updated by The Foreman Bot 8 months ago

  • Assignee set to Ewoud Kohl van Wijngaarden
  • Status changed from New to Ready For Testing
  • Pull request https://github.com/theforeman/puppet-foreman/pull/800 added

#2 Updated by The Foreman Bot 8 months ago

  • Pull request https://github.com/theforeman/puppet-foreman_proxy_content/pull/243 added

#3 Updated by The Foreman Bot 8 months ago

  • Pull request https://github.com/theforeman/puppet-katello/pull/324 added

#4 Updated by The Foreman Bot 8 months ago

  • Fixed in Releases 2.1.0 added

#5 Updated by Ewoud Kohl van Wijngaarden 8 months ago

  • Status changed from Ready For Testing to Closed

#6 Updated by The Foreman Bot 8 months ago

  • Pull request https://github.com/theforeman/puppet-katello_devel/pull/225 added

Also available in: Atom PDF