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

backports: don't hardcode an old gpg key for Ubuntu #1129

Merged
merged 3 commits into from
Nov 21, 2023

Conversation

kenyon
Copy link

@kenyon kenyon commented Sep 18, 2023

Summary

Same as #847 but for Ubuntu.

The gpg keys used to sign the Ubuntu repos, including backports, come with standard Ubuntu installations, so this Puppet module doesn't need to add any key.

Having this old key causes Puppet run failures for 20.04 and newer machines without Internet access, since it tries to fetch the key from the Internet.

Related Issues (if any)

#847

@kenyon kenyon requested review from bastelfreak, smortex and a team as code owners September 18, 2023 19:39
@kenyon kenyon force-pushed the ubuntu-backport-gpg-key branch from 445e5f2 to f768a62 Compare September 18, 2023 21:02
@kenyon
Copy link
Author

kenyon commented Sep 18, 2023

Not sure why that Ubuntu 18.04 test is failing, but it started failing in #1124, and seems unrelated to this module's code.

Copy link
Collaborator

@smortex smortex left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This look fine. CI failure looks unrelated.

The default key for Debian was removed in puppetlabs#847, and I'm about to do the
same for Ubuntu.
Same as puppetlabs#847 but for Ubuntu.

The gpg keys used to sign the Ubuntu repos, including backports, come
with standard Ubuntu installations, so this Puppet module doesn't need
to add any key.

Having this old key causes Puppet run failures for 20.04 and newer
machines without Internet access.
@kenyon kenyon force-pushed the ubuntu-backport-gpg-key branch from f768a62 to f867561 Compare November 1, 2023 04:53
@bastelfreak
Copy link
Collaborator

@kenyon let me know if bug label isn't correct.

@bastelfreak bastelfreak merged commit bee5ee1 into puppetlabs:main Nov 21, 2023
16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants