Skip to content

Add alternative location for GPG keys for the zabbix and unsupported #987

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

Merged
merged 3 commits into from
Feb 20, 2025

Conversation

FunPat
Copy link
Contributor

@FunPat FunPat commented Feb 18, 2025

Pull Request (PR) description

Added two variables to allow users of this module to set an alternative resource location for the GPG keys, when operating in an airlocked environment where servers cannot talk out to repo.zabbix.com directly DNF/YUM will fail to check the GPG key check if the repositories are managed by this module. This will allow an internal GPG key to be referenced so that the puppet-zabbix module can be used on servers that cannot have direct internet access, this is especially important from an agent perspective.

Variables introduced:

  • repo_gpg_key_location
  • unsupported_repo_gpg_key_location

Two checks have been introdcued under the repo.spec for RedHat to test that when these have been set that the GPG key has the alternative location set. Used https://example.com/bar to represent if it is a different location to where the repository is, ie repository location being https://example.com/foo.

This Pull Request (PR) fixes the following issues

No current issues relating to this that I'm aware of.

…repositories to allow for this to be deployed in an airlocked environment using a reposync on a resource server
@FunPat
Copy link
Contributor Author

FunPat commented Feb 19, 2025

Ready for review when someone is available, thanks in advance. Any questions, concerns please let me know.
Love what the voxpupli community do! I hope this is a valuable addition to the module, would certainly assist in the environment I work in :)

@evgeni evgeni added the enhancement New feature or request label Feb 20, 2025
@evgeni evgeni merged commit 383c15e into voxpupuli:master Feb 20, 2025
74 checks passed
@FunPat FunPat mentioned this pull request Jun 12, 2025
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