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

fix(release): another attempt to fix the release step #531

Merged
merged 1 commit into from
May 8, 2024

Conversation

transient1
Copy link
Contributor

changing {artifact} in signs cmd to {digest}

This PR fixes #

Checklist

  • I have signed the CLA
  • I have updated/added any relevant documentation

Description

What's the goal of this PR?

Fix the release step which is currently fails with error message

error=signing docker images: failed to publish artifacts: sign: cosign failed: exit status 1: WARNING: Image reference sha256:424fa83f860fe96d1966c0d1cab7c62fc06d36a2e763bad1d8ce004cca306ada uses a tag, not a digest, to identify the image to sign.

and

Error: signing [sha256:424fa83f860fe96d1966c0d1cab7c62fc06d36a2e763bad1d8ce004cca306ada]: accessing entity: GET https://index.docker.io/v2/library/sha256/manifests/424fa83f860fe96d1966c0d1cab7c62fc06d36a2e763bad1d8ce004cca306ada: UNAUTHORIZED: authentication required; [map[Action:pull Class: Name:library/sha256 Type:repository]]
main.go:74: error during command execution: signing [sha256:424fa83f860fe96d1966c0d1cab7c62fc06d36a2e763bad1d8ce004cca306ada]: accessing entity: GET https://index.docker.io/v2/library/sha256/manifests/424fa83f860fe96d1966c0d1cab7c62fc06d36a2e763bad1d8ce004cca306ada: UNAUTHORIZED: authentication required; [map[Action:pull Class: Name:library/sha256 Type:repository]]

What changes did you make?

${artifact} to ${digest} in signs section

What alternative solution should we consider, if any?

changing `{artifact}` in `signs` cmd to `{digest}`
@transient1 transient1 requested a review from sudermanjr as a code owner May 8, 2024 16:02
Copy link

Fairwinds Insights Scan Results

View the full report at insights.fairwinds.com.

✅ No new Action Items detected!

@transient1 transient1 merged commit 833dc21 into master May 8, 2024
9 checks passed
@transient1 transient1 deleted the sc/change-signing branch May 8, 2024 16:08
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