Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Allow this module to work on systems when global repo_gpgcheck = 1 #82

Merged
merged 6 commits into from
Feb 6, 2020

Conversation

hammondr
Copy link

Some system hardening guides require the use of "repo_gpgcheck = 1" in yum.conf. This setting breaks this puppet module because the repo files it generates cannot set a repo-specific repo_gpgcheck setting (i.e. set it to 0 for epel).

This patch sets a value of 0 for each supported epel repo and allows override via params/hiera.

@rnelson0
Copy link
Member

rnelson0 commented Apr 26, 2019

It seems like the default is 1, not 0, as you describe. I believe it should be disabled by default based on my understanding of both the flag and the amount of metadata RH signs, but I could be wrong there (appears to have only been fixed with 7.4, did 6 repos get backported signatures?)

@hammondr
Copy link
Author

Neither EPEL 6 nor 7 contain signed repo metadata. Ref:

This patch makes sure repo_gpgcheck is disabled by default for the EPEL repo. For hardened systems with "repo_gpgcheck=1" in yum.conf, the EPEL repo will no longer break. Systems that aren't enforcing "repo_gpgcheck=1" in yum.conf will see no behavior changes.

@rnelson0
Copy link
Member

Ah, so the local setting can override the global, makes sense. Sorry, it was early, I'm get it now :D

@rnelson0 rnelson0 self-requested a review April 26, 2019 15:34
@hammondr
Copy link
Author

hammondr commented Jun 6, 2019

Thanks for the approval. Are there any more steps before you can merge this?

@rnelson0 rnelson0 merged commit 0771f2c into voxpupuli:master Feb 6, 2020
@rnelson0
Copy link
Member

rnelson0 commented Feb 6, 2020

I am horribly sorry this fell off my radar for so long! I've resolved the merge conflicts and merged this PR. Thank you so much for your patience.

@rnelson0 rnelson0 added the enhancement New feature or request label Feb 6, 2020
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants