Warning: Setting config_version is deprecated in puppet.conf.
The "config_version" needs to be able to be "left out" of the puppet.conf config file conditionally (probably 3.6+)
At a minimum, server_config_version should be able to be set to null or "" (empty) to get it to exclude like the "environment" template has. Automating its presence based on puppet version is probably a good thing (tm).
Here is the deprecation warning:
Warning: Setting config_version is deprecated in puppet.conf. See http://links.puppetlabs.com/env-settings-deprecations
(at /usr/share/ruby/vendor_ruby/puppet/settings.rb:1141:in `issue_deprecation_warning')
#3 Updated by Tommy McNeely almost 7 years ago
No, I have it set to false because my "puppet-environments" repo uses a "Puppetfile" and "r10k" to deploy it. From what I can tell (and I may be wrong) the "git environment" expects everything to be in each repo branch.
However, I agree that these two issues are related, and perhaps need to just become a CR back to Puppet asking them what method we should use to set the config_version in a dynamic environment going forward?
#4 Updated by Dominic Cleal almost 7 years ago
My understanding is that we now need to populate environment.conf for each environment to set it, there's no global method. For the git environments case we probably need to enhance the git hook, and for your r10k environment you probably need to add environment.conf to the managed files.
#6 Updated by Tommy McNeely almost 7 years ago
Chatting with Dominic and gwmngilfen we found the problem...
I had originally set:
... because I am indeed using dynamic environments, but that actually was a setting that should be false because I am using directory environments. So it should be:
That doesn't necessarily mean that there shouldn't be some sort of "conflicting options" checks or "version" check here, but its far less critical ;)
#7 Updated by Dominic Cleal over 6 years ago
- Status changed from New to Resolved
https://github.com/theforeman/puppet-puppet/commit/97b6a6f1bd0e18c19fe52a92232914debafbb4ef should resolve any stray config_version settings being added too when directory environments are enabled.