Skip to content

Conversation

@techsk8
Copy link

@techsk8 techsk8 commented Jun 27, 2025

Fixes

W: https://artifacts.elastic.co/packages/7.x/apt/dists/stable/InRelease: Key is stored in legacy trusted.gpg keyring (/etc/apt/trusted.gpg), see the DEPRECATION section in apt-key(8) for details.

@techsk8 techsk8 force-pushed the manage_gpgkey_with_aptkeyring branch from 9175a3e to bb1d5fc Compare June 27, 2025 22:21
@techsk8 techsk8 force-pushed the manage_gpgkey_with_aptkeyring branch from bb1d5fc to 3ce9cb4 Compare June 27, 2025 22:22
@smortex smortex requested a review from h-haaks June 27, 2025 22:43
Copy link
Member

@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.

LGTM, added another reviewer as I do not use these modules and might miss something.

@techsk8 techsk8 requested a review from smortex July 15, 2025 03:08
@techsk8
Copy link
Author

techsk8 commented Aug 26, 2025

any news?

@h-haaks
Copy link
Contributor

h-haaks commented Sep 5, 2025

Hi @techsk8

As far as I know the code already create the keyring file in /etc/apt/keyrings/elastic-keyring.asc.
Ref https://github.com/puppetlabs/puppetlabs-apt/blob/main/manifests/source.pp#L13

I'm afraid managing apt::keyring in this module won't solve your issue ..

Are you having this issue on a fresh/new server or are you upgrading a server that already has the old gpg key installed?

@h-haaks
Copy link
Contributor

h-haaks commented Sep 5, 2025

could you please share the content of /etc/apt/sources.list.d/elastic.list and the output of apt-key list

@h-haaks
Copy link
Contributor

h-haaks commented Sep 5, 2025

I suspect your issue is related to module upgrade 9.x -> 10.0.0
9.x adds the pgp-key in /etc/apt/trusted.gpg with key_id '46095ACC8548582C1A2699A9D27D666CD88E42B4'
10.0.0 adds the keyring in /etc/apt/keyrings/elastic-keyring.asc but does not remove the key in /etc/apt/trusted.gpg

If I'm right `sudo apt-key del 46095ACC8548582C1A2699A9D27D666CD88E42B4' should solve this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants