Skip to content

Add an option to invalidate GPG keys#880

Merged
bstansberry merged 1 commit intowildfly:mainfrom
jamezp:invalidated-pgp-keys
Mar 23, 2026
Merged

Add an option to invalidate GPG keys#880
bstansberry merged 1 commit intowildfly:mainfrom
jamezp:invalidated-pgp-keys

Conversation

@jamezp
Copy link
Copy Markdown
Member

@jamezp jamezp commented Mar 10, 2026

No description provided.

@github-actions
Copy link
Copy Markdown

A preview of this PR is available at https://pr-880-wildfly-org.surge.sh

link: https://keys.openpgp.org/vks/v1/by-fingerprint/51F8871824A19B8C3EE6708AC34DC24F80C9A5BA
fingerprint: 51F8 8718 24A1 9B8C 3EE6 708A C34D C24F 80C9 A5BA
invalidated: 2026-03-10
invalidated-reason: No longer available
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.

I've left this free-form, but I'm not sure what we want to do here. We could have defined reasons or leave it free-formed like this. "No longer available" might also not be a great reason :)

@@ -0,0 +1,54 @@
/*
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.

Why add this and delete Contributors.java? Roq should handle all of this for us.

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.

I got errors with the use of java.time.LocalDate. There is a chance I did something wrong, but I got errors with the BytecodeRecorderImpl.

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.

Seems to be a long-standing issue. I've filed an issue upstream: quarkiverse/quarkus-roq#779

Signed-off-by: James R. Perkins <jperkins@ibm.com>
@jamezp jamezp force-pushed the invalidated-pgp-keys branch from 82bd76c to a0c5a8f Compare March 10, 2026 22:07
Copy link
Copy Markdown
Contributor

@bstansberry bstansberry left a comment

Choose a reason for hiding this comment

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

I think you should remove the contributors.yaml change. We should merge this and then we should ask Pedro to update and sign his commit.

Let's avoid edits to these records by people other than the key owner.

@jamezp
Copy link
Copy Markdown
Member Author

jamezp commented Mar 10, 2026

No problem. I'll remove it now. Part of adding was to see what it would look like, but it makes sense that the owner of the key should change it.

@jamezp jamezp force-pushed the invalidated-pgp-keys branch from a0c5a8f to 2d35013 Compare March 10, 2026 22:28
@bstansberry
Copy link
Copy Markdown
Contributor

yeah, using it to test the result made sense

@bstansberry bstansberry mentioned this pull request Mar 10, 2026
@jamezp jamezp changed the title Add an option to invalidate GPG codes and update the GPG code for Pedro Add an option to invalidate GPG keys Mar 10, 2026
@bstansberry bstansberry merged commit 8b8cb68 into wildfly:main Mar 23, 2026
1 check passed
@bstansberry
Copy link
Copy Markdown
Contributor

Thanks @jamezp and @jasondlee

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