Added class param to disable YUM repo management on RedHat platforms #144
Conversation
|
I've suggested something similar in #145 for Debian: add parameter |
|
Would it also allow to disable repository management altogether? For example in my environment the repositories aren't managed by Puppet at all, so it would be nice to be able to disable this functionality. |
|
This is a good pattern. I'd like to use this for sure. |
| gpgcheck => '1', | ||
| priority => '1', | ||
| gpgkey => 'http://nginx.org/keys/nginx_signing.key', | ||
| if $nginx::manage_repo { |
There was a problem hiding this comment.
What's the reasoning for calling the out-of-scope variable?
I almost prefer an tell-don't-ask approach versus ask-don't-tell. Thoughts?
…tory should be managed
|
Makes sense, I've now patched it so that the param is pushed down all the way to the package.pp and package/redhat.pp. |
|
Awesome. Thanks so much! |
I've added an extra class param to disable repository management. Right now the code only has been modified for RedHat based platforms.