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

Publish cosign bundle alongside qcow2 files #143

Merged
merged 10 commits into from
Feb 14, 2024
Merged

Publish cosign bundle alongside qcow2 files #143

merged 10 commits into from
Feb 14, 2024

Conversation

JohnGarbutt
Copy link
Contributor

No description provided.

@JohnGarbutt JohnGarbutt marked this pull request as ready for review February 12, 2024 09:28
@JohnGarbutt JohnGarbutt requested review from mkjpryor and a team as code owners February 12, 2024 09:28
@JohnGarbutt JohnGarbutt changed the title Use cosign on the qcow2 files Publish cosign bundle alongside qcow2 files Feb 12, 2024
Copy link
Collaborator

@mkjpryor mkjpryor left a comment

Choose a reason for hiding this comment

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

We need to modify the manifest generation so that the cosign bundle URL is included and the purge scripts need modifying to also remove the cosign bundles.

@JohnGarbutt JohnGarbutt requested a review from mkjpryor February 13, 2024 09:24
bin/purge-defunct Outdated Show resolved Hide resolved
@JohnGarbutt JohnGarbutt added the enhancement New feature or request label Feb 13, 2024
Copy link
Collaborator

@mkjpryor mkjpryor left a comment

Choose a reason for hiding this comment

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

LGTM

@JohnGarbutt JohnGarbutt merged commit a5765cb into main Feb 14, 2024
14 checks passed
@JohnGarbutt JohnGarbutt deleted the sign-qcow2 branch February 14, 2024 09:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants