Skip to content

signing-keys: add Fedora 45 key#1722

Merged
dustymabe merged 2 commits intocoreos:mainfrom
aaradhak:f44key
Feb 10, 2026
Merged

signing-keys: add Fedora 45 key#1722
dustymabe merged 2 commits intocoreos:mainfrom
aaradhak:f44key

Conversation

@aaradhak
Copy link
Copy Markdown
Member

@aaradhak aaradhak commented Feb 7, 2026

Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request adds the signing key for Fedora 45 and updates the release notes. The change is straightforward, but it appears to be incomplete. Based on the project's history, adding a new key is usually accompanied by removing the oldest one. I've suggested removing the Fedora 42 key and updating the release notes to reflect this, to maintain the key rotation policy.


Major changes:

- Add Fedora 45 signing key
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

medium

Based on the project's key rotation policy, when a new Fedora key is added, the oldest one is typically removed. With the addition of the Fedora 45 key, the Fedora 42 key should also be removed from src/signing-keys.asc. Please update this release note to reflect that.

Suggested change
- Add Fedora 45 signing key
- Add Fedora 45 signing key; drop Fedora 42 signing key

Copy link
Copy Markdown
Member

@dustymabe dustymabe left a comment

Choose a reason for hiding this comment

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

LGTM

@dustymabe
Copy link
Copy Markdown
Member

This is blocked on coreos/coreos-assembler#4431

@dustymabe
Copy link
Copy Markdown
Member

Also let's fix CI here with #1724

@dustymabe dustymabe enabled auto-merge (rebase) February 10, 2026 00:02
@dustymabe dustymabe merged commit 2443460 into coreos:main Feb 10, 2026
15 checks passed
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.

2 participants