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

Don't set config_version to server_config_version for every environment #132

Closed

Conversation

mcanevet
Copy link
Contributor

There is no need to default config_version to server_config_version for puppet::server::env because the default behavior is to take the config_version from the master section

@domcleal
Copy link
Contributor

I haven't verified the default behaviour that @mcanevet mentioned, but otherwise looks fine.

@ekohl
Copy link
Member

ekohl commented Mar 13, 2014

Do we have the config_version by default in master? If so, 👍

@domcleal
Copy link
Contributor

Hmm, seems we don't. It's only set when using git or dynamic environments:

https://github.com/theforeman/puppet-puppet/blob/master/templates/server/puppet.conf.erb#L17-L21

Whereas in server::env it's always added if the parameter is set.

@mcanevet
Copy link
Contributor Author

Not relevant anymore. I close.

@mcanevet mcanevet closed this Sep 24, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants