Skip to content
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

Respect puppet::server::additional_settings parameter #874

Merged
merged 1 commit into from
May 16, 2023

Conversation

ekohl
Copy link
Member

@ekohl ekohl commented May 16, 2023

Previously it was read from the main puppet class, instead of puppet::server (which in turn defaulted to the parameter from the main class).

Fixes #810

Previously it was read from the main puppet class, instead of
puppet::server (which in turn defaulted to the parameter from the main
class).
@ekohl
Copy link
Member Author

ekohl commented May 16, 2023

I think the idempotency issue is just a fluke and #874 drops Puppet 6 support anyway.

@ekohl ekohl merged commit b36debb into theforeman:master May 16, 2023
@ekohl ekohl deleted the correct-variable branch May 16, 2023 12:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

server::config class references wrong variable for additional_settings
2 participants