-
Notifications
You must be signed in to change notification settings - Fork 270
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Refactor Puppet handling #662
Conversation
lint fails |
Looks like I accidentally removed a bunch of code that would have broken otherwise. That was not intentional. |
This should wait on theforeman/foreman#5588 I guess? |
I'm not sure. While that variable is used by Foreman to determine defaults for other values, we don't really use the value because ssl certs are explicitly configured. That's why I'm slightly in doubt about it. |
This moves all puppetmaster defaults to its own parameters file. Parameters that were used in both are duplicated. It also adds Puppet types to all parameters. This removes puppet settings from the top level. The puppet_ssldir is now an internal detail to determine defaults. It is also no longer set in Foreman since it's only used as a base for determining SSL paths which we explicitly set. There is a PR open to remove this setting altogether.
@mmoll do you want to wait for the core merge or can we merge this now? IMHO both are ready to be merged now. |
merged, bedankt @ekohl! |
This moves all puppetmaster defaults to its own parameters file.
Parameters that were used in both are duplicated. It also adds Puppet
types to all parameters.
This removes puppet settings from the top level. The puppet_ssldir is
now an internal detail to determine defaults. It is also no longer set
in Foreman since it's only used as a base for determining SSL paths
which we explicitly set. There is a PR open to remove this setting
altogether.