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

(MODULES-10971) - Ensure apt::keyserver is considered when creating a default apt:source #981

Conversation

david22swan
Copy link
Member

@david22swan david22swan commented Mar 24, 2021

As it stands the source_key_defaults point towards the default keyserver value set within the params. This change makes it so that it will instead use the keyserver value set in the init, which will in turn default to the params value if none has been given.

@puppet-community-rangefinder
Copy link

apt::params is a class

that may have no external impact to Forge modules.

This module is declared in 234 of 576 indexed public Puppetfiles.


These results were generated with Rangefinder, a tool that helps predict the downstream impact of breaking changes to elements used in Puppet modules. You can run this on the command line to get a full report.

Exact matches are those that we can positively identify via namespace and the declaring modules' metadata. Non-namespaced items, such as Puppet 3.x functions, will always be reported as near matches only.

@david22swan david22swan force-pushed the MODULES-10971/main/aptkeyserver_fix branch from 4530429 to 18522b9 Compare March 24, 2021 16:24
… a default apt:source

As it stands the source_key_defaults point towards the default `keyserver` value set within the params. This change makes it so that it will instead use the `keyserver` value set in the init, which will in turn default to the params value in none has been given.
@david22swan david22swan force-pushed the MODULES-10971/main/aptkeyserver_fix branch from 18522b9 to a6eb161 Compare March 24, 2021 16:26
@david22swan david22swan marked this pull request as ready for review March 24, 2021 16:49
@david22swan david22swan requested a review from a team as a code owner March 24, 2021 16:49
@michaeltlombardi michaeltlombardi merged commit 15277f3 into puppetlabs:main Mar 25, 2021
@david22swan david22swan deleted the MODULES-10971/main/aptkeyserver_fix branch March 25, 2021 13:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants