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

Fix default PGP key #308

Merged
merged 2 commits into from
Feb 3, 2023
Merged

Fix default PGP key #308

merged 2 commits into from
Feb 3, 2023

Conversation

smortex
Copy link
Member

@smortex smortex commented Feb 3, 2023

According to https://packages.grafana.com/:

The GPG key used to sign the APT repository (fingerprint
4E40DDF6D76E284A4A6780E48C8C34C524098CB6) was rotated on 2023-01-12 and
replaced with a new key with fingerprint
0E22EB88E39E12277A7760AE9E439B102CF3C0C6.

Update the key accordingly.

@@ -53,7 +53,7 @@
# Defaults to https://packages.grafana.com/gpg.key.
#
# @param repo_key_id When using 'repo' install_method, the repo_key_id of the repo_gpg_key_url key on Debian based systems.
# Defaults to 4E40DDF6D76E284A4A6780E48C8C34C524098CB6.
# Defaults to 0E22EB88E39E12277A7760AE9E439B102CF3C0C6.
Copy link
Member Author

Choose a reason for hiding this comment

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

The default value should be read from the actual code, for now just update it: there is a lot of parameter that read a value from data/common.yaml and these default parameters should be moved around in another PR.

@smortex smortex added the bug Something isn't working label Feb 3, 2023
@smortex
Copy link
Member Author

smortex commented Feb 3, 2023

So Debian CI fails because the Grafana repos are broken (metadata reference a package version that is not available, maybe transcient), Ubuntu CI fails because it needs the same breakage as Debian, and CentOS 8 CI fails because CentOS sucks. EVERYTHING IS FINE!

It least I could check this successfully in the real world! Ready for review!

@smortex smortex marked this pull request as ready for review February 3, 2023 18:33
According to https://packages.grafana.com/:
> The GPG key used to sign the APT repository (fingerprint
> 4E40DDF6D76E284A4A6780E48C8C34C524098CB6) was rotated on 2023-01-12 and
> replaced with a new key with fingerprint
> 0E22EB88E39E12277A7760AE9E439B102CF3C0C6.

Update the key accordingly.
@smortex smortex linked an issue Feb 3, 2023 that may be closed by this pull request
@smortex smortex merged commit 353f7b2 into master Feb 3, 2023
@smortex smortex deleted the fix-apt-repo branch February 3, 2023 23:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

repository setup: new GPG repository keys
2 participants