Skip to content

docs: include PGP key fingerprints#13112

Merged
babakks merged 3 commits intotrunkfrom
babakks/add-pgp-fingerprints-to-install-docs
Apr 8, 2026
Merged

docs: include PGP key fingerprints#13112
babakks merged 3 commits intotrunkfrom
babakks/add-pgp-fingerprints-to-install-docs

Conversation

@babakks
Copy link
Copy Markdown
Member

@babakks babakks commented Apr 7, 2026

This PR adds fingerprints of our PGP keys to Linux installation docs.

Signed-off-by: Babak K. Shandiz <babakks@github.com>
Copilot AI review requested due to automatic review settings April 7, 2026 21:14
@babakks babakks requested a review from a team as a code owner April 7, 2026 21:14
@babakks babakks requested a review from BagToad April 7, 2026 21:14
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds PGP key fingerprints to the Linux installation documentation so users can confirm the signing keys used by the official package repositories.

Changes:

  • Adds an IMPORTANT callout listing the PGP key fingerprints used for signing.
  • Adds guidance for verifying the Debian APT keyring contents with gpg --show-keys.
  • Adds an RPM section TIP reminding users they can validate imported keys against the published fingerprints.
Show a summary per file
File Description
docs/install_linux.md Documents signing key fingerprints and adds key verification tips for Debian/RPM installs.

Copilot's findings

Tip

Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

  • Files reviewed: 1/1 changed files
  • Comments generated: 2

Comment thread docs/install_linux.md Outdated
Comment thread docs/install_linux.md
Signed-off-by: Babak K. Shandiz <babakks@github.com>
Comment thread docs/install_linux.md Outdated
> To verify downloaded PGP keys, you can run this and match the listed fingerprints with those at the top of this document:
>
> ```shell
> gpg --show-keys /etc/apt/keyrings/githubcli-archive-keyring.gpg
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Is there a reason you didn't include this for RPM?

Copy link
Copy Markdown
Member Author

@babakks babakks Apr 8, 2026

Choose a reason for hiding this comment

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

Yeah. Seems like RPM clients download the keyring file and then install it as an RPM package. The original .asc file should be somewhere on the disk, but the source of truth is rpm -qi query.

Nevermind, misread/confused. Yeah, I actually can do a curl/wget and then run gpg on it. Will add now.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Now added for both, but I changed my mind a bit. Instead of verifying the downloaded/installed keyring, now we verify the keyrings to be downloaded (via curl | gpg). It's actually more realistic for users who want to check the fingerprints first.

Signed-off-by: Babak K. Shandiz <babakks@github.com>
@babakks babakks force-pushed the babakks/add-pgp-fingerprints-to-install-docs branch from 6b4b344 to 1f94ee4 Compare April 8, 2026 12:07
@babakks babakks merged commit 97ba17b into trunk Apr 8, 2026
6 checks passed
@babakks babakks deleted the babakks/add-pgp-fingerprints-to-install-docs branch April 8, 2026 13:55
@t72053166-eng t72053166-eng mentioned this pull request Apr 17, 2026
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