Refactor #23635: Optionally load facter - Foreman

Refactor #23635

Optionally load facter

Added by Ewoud Kohl van Wijngaarden 4 months ago. Updated about 2 months ago.

Status:

Closed

Priority:

Normal

Assignee:

Ewoud Kohl van Wijngaarden

Category:

Settings

Target version:

1.19.0

Difficulty:

Triaged:

Bugzilla link:

Pull request:

https://github.com/theforeman/foreman/pull/5589

Team Backlog:

Fixed in Releases:

1.19.0

Found in Releases:

Description

Facter is currently loaded in a few places. This is always to get the fqdn or domain which can be centralized to settings. It can then detect if those are already hardcoded. The benefit is that facter becomes optional.

Associated revisions

Revision c1f2c0bb (diff)
Added by Ewoud Kohl van Wijngaarden 4 months ago

Refs #23635 - Stop relying on facter

The inclusion of facter is technically optional since it's a bundler
group. In practice there are direct calls to it.

This change relies on a change in Foreman core to ensure SETTINGS[:fqdn]
is set.

The benefit is that you can explicitly set the FQDN in your settings
file and avoid the use of the host FQDN altogether. This has the benefit
of decoupling from the actual host you run Foreman on.

Revision a653e389 (diff)
Added by Ewoud Kohl van Wijngaarden 4 months ago

Fixes #23635 - Centralize use of facter

We have a bundler group for facter. That implies it's optional but
before this change it wasn't With this change it's only required if no
domain and fqdn are set. This is still the default so in most cases it's
no difference, but at least we have centralized all use of facter to one
place.