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

Add ability to set gpgcheck for yum repos #438

Merged
merged 1 commit into from
Aug 18, 2021

Conversation

dingus9
Copy link
Contributor

@dingus9 dingus9 commented Aug 12, 2021

SUMMARY

Add the ability and property(zabbix_repo_yum_gpgcheck) to control gpg key checking on the default yum repo entries

This feature should allow users to more easily toggle gpg key checking with the default yum repo values

ISSUE TYPE
  • Feature Pull Request
COMPONENT NAME

improvement to role

@dj-wasabi
Copy link
Contributor

Hi @dingus9

Thank you for making time to create a PR and provide a improvement. Can I ask you if you can apply the same change to the other roles as well, so we have a consistent way of configuring repo's? Other people that has both the Agent and Server/Proxy on the same node will have no idempotency, as each role is changing the contents of the repo files.

Thank you!

@dj-wasabi dj-wasabi added the enhancement New feature or request label Aug 13, 2021
@dingus9
Copy link
Contributor Author

dingus9 commented Aug 13, 2021

Sure I'll look into that change on the others also

@dingus9 dingus9 changed the title Add ability to set gpgcheck for yum repos WIP: Add ability to set gpgcheck for yum repos Aug 13, 2021
@dingus9 dingus9 changed the title WIP: Add ability to set gpgcheck for yum repos Add ability to set gpgcheck for yum repos Aug 17, 2021
@dingus9
Copy link
Contributor Author

dingus9 commented Aug 17, 2021

Does that look about right? Not sure if I got them all!

@dj-wasabi dj-wasabi merged commit ed0dd33 into ansible-collections:main Aug 18, 2021
@dj-wasabi
Copy link
Contributor

Hi @dingus9

Yes, looks very good! Thank you! 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants