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

CASMINST-6896: Support multiple GPG signing keys and install them #279

Merged
merged 4 commits into from
Jun 26, 2024

Conversation

jpdavis-prof
Copy link
Contributor

@jpdavis-prof jpdavis-prof commented Jun 26, 2024

Summary and Scope

Add multiple GPG keys via CFS
Most of this code was written by @mtupitsyn, I just borrowed it.

Issues and Related PRs

Testing

node-personalization and image customization were both successful

Tested on:

  • tyr

Risks and Mitigations

none known

Pull Request Checklist

  • Version number(s) incremented, if applicable
  • Copyrights updated
  • License file intact
  • Target branch correct
  • CHANGELOG.md updated
  • Testing is appropriate and complete, if applicable

@jpdavis-prof jpdavis-prof requested a review from a team as a code owner June 26, 2024 15:00
@mharding-hpe mharding-hpe changed the title Support multiple GPG signing keys and install them on NCNs CASMINST-6896: Support multiple GPG signing keys and install them Jun 26, 2024
Update version index at bottom of changelog

(cherry picked from commit c53ebb5)
Update copyright dates

(cherry picked from commit 3918663)
Copy link
Contributor

@mharding-hpe mharding-hpe left a comment

Choose a reason for hiding this comment

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

I'm marking this as "requesting changes" to make sure that this doesn't merge until @mtupitsyn 's concern has been addressed.

ansible/roles/csm.gpg_keys/tasks/main.yml Show resolved Hide resolved
@mharding-hpe mharding-hpe merged commit 9e21e94 into release/1.16 Jun 26, 2024
8 checks passed
@mharding-hpe mharding-hpe deleted the CASMINST-6896-1.5 branch June 26, 2024 17:42
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.

4 participants