fix: attempt to fix cosign compatibility issues#5959
Conversation
|
Note Reviews pausedIt looks like this branch is under active development. To avoid overwhelming you with review comments due to an influx of new commits, CodeRabbit has automatically paused this review. You can configure this behavior by changing the Use the following commands to manage reviews:
Use the checkboxes below for quick actions:
WalkthroughBumps cosign v3.0.3→v3.0.4, adds signing/attestation flags (--new-bundle-format=false, --use-signing-config=false), restructures digest/signing/SBOM/provenance steps into grouped log blocks, loosens certificate identity regexp, updates docker/setup-buildx-action pin, and removes an explicit buildx version env var. Changes
Possibly related PRs
🚥 Pre-merge checks | ✅ 1✅ Passed checks (1 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Signed-off-by: Gustavo Carvalho <gustavo.carvalho@container-solutions.com>
12c2186 to
2200b27
Compare
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Fix all issues with AI agents
In @.github/actions/sign/action.yml:
- Around line 72-76: The cosign workflow uses --new-bundle-format=false for
cosign sign but not for cosign attest, risking mixed bundle formats; update
every cosign attest invocation (the commands invoking "cosign attest") to
include --new-bundle-format=false so all attestation commands match the sign
command's bundle format flag and ensure consistent artifact format across the
action.
Signed-off-by: Gustavo Carvalho <gustavo.carvalho@container-solutions.com>
Signed-off-by: Gustavo Carvalho <gustavo.carvalho@container-solutions.com>
Signed-off-by: Gustavo Carvalho <gustavo.carvalho@container-solutions.com>
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Fix all issues with AI agents
In @.github/actions/sign/action.yml:
- Line 114: The cosign verify-attestation command invocation contains a
duplicated flag and a missing space: remove the extra duplicated
"--new-bundle-format=false" and ensure there is a space before "--type spdx" so
the command reads like a single "--new-bundle-format=false --type spdx
${IMAGE_NAME}@${CONTAINER_DIGEST}" invocation; update the line that builds the
cosign command (the cosign verify-attestation call) to contain only one
"--new-bundle-format=false" followed by a spaced "--type spdx" and the existing
${IMAGE_NAME}@${CONTAINER_DIGEST} token.
Signed-off-by: Gustavo Carvalho <gustavo.carvalho@container-solutions.com>
Signed-off-by: Gustavo Carvalho <gustavo.carvalho@container-solutions.com>
Signed-off-by: Gustavo Carvalho <gustavo.carvalho@container-solutions.com>
|
Oh gods it failed to horribly. :D |
Signed-off-by: Gustavo Carvalho <gustavo.carvalho@container-solutions.com>
almost there. Some bits failed, but the general idea is correct. One of the tries succeeded in signing and attesting an image, just failed verify-attestation for some reason (need to check / study that eventually). I think this run should work :) |
…despite flags) Signed-off-by: Gustavo Carvalho <gustavo.carvalho@container-solutions.com>
Signed-off-by: Gustavo Carvalho <gustavo.carvalho@container-solutions.com>
Signed-off-by: Gustavo Carvalho <gustavo.carvalho@container-solutions.com>
|
| uses: sigstore/cosign-installer@faadad0cce49287aee09b3a48701e75088a2c6ad # v4.0.0 | ||
| with: | ||
| cosign-release: 'v3.0.3' | ||
| cosign-release: 'v3.0.2' |
There was a problem hiding this comment.
the PR description says it upgrades to v3.0.4 (which is latest from January this year), but in fact it's downgraded to v3.0.2. I didn't check the commit history as to why it's needed to downgrade it. is it really?
There was a problem hiding this comment.
We needed to revert because there is a regression on 3.0.4 vs 3.0.2 (don’t know exactly why) - co-sign verify attestation is forcing the checks with the new bundle format all the times (and failing)
this is fairly easy to reproduce locally (once we know it’s a thing)
I didn’t test 3.0.3 though
|
🚀 LGTM |
moolen
left a comment
There was a problem hiding this comment.
LGTM. No need to block PR for version bump
|
We needed to revert because there is a regression on 3.0.4 vs 3.0.2 (don’t know exactly why) - co-sign verify attestation is forcing the checks with the new bundle format all the times (and failing) this is fairly easy to reproduce locally (once we know it’s a thing) |
|
So... is this now mergeable? |
* fix: attempt to fix ci pipeline Signed-off-by: Gustavo Carvalho <gustavo.carvalho@container-solutions.com> * fix: e2e tests with old docker buildx Signed-off-by: Gustavo Carvalho <gustavo.carvalho@container-solutions.com> * fix: coderabbit issues -- thanks coderabbit Signed-off-by: Gustavo Carvalho <gustavo.carvalho@container-solutions.com> * fix: force old bundle format for verify-attestation Signed-off-by: Gustavo Carvalho <gustavo.carvalho@container-solutions.com> * fix: typo Signed-off-by: Gustavo Carvalho <gustavo.carvalho@container-solutions.com> * fix: use-signing-config to false Signed-off-by: Gustavo Carvalho <gustavo.carvalho@container-solutions.com> * fix: verify attestation does not use signing config Signed-off-by: Gustavo Carvalho <gustavo.carvalho@container-solutions.com> * fix: readd arm64 for -ubi-boringssl Signed-off-by: Gustavo Carvalho <gustavo.carvalho@container-solutions.com> * fix: remove bundle format from verify attestation Signed-off-by: Gustavo Carvalho <gustavo.carvalho@container-solutions.com> * fix: cosign regression on 3.0.4 prevents old pipelines from working (despite flags) Signed-off-by: Gustavo Carvalho <gustavo.carvalho@container-solutions.com> * fix: ci bloat and provenance version Signed-off-by: Gustavo Carvalho <gustavo.carvalho@container-solutions.com> * fix: replace slsa-provenance-action with custom bash Signed-off-by: Gustavo Carvalho <gustavo.carvalho@container-solutions.com> --------- Signed-off-by: Gustavo Carvalho <gustavo.carvalho@container-solutions.com> Signed-off-by: Nattapong Ekudomsuk <nuttapong_mos@hotmail.com>
* fix: attempt to fix ci pipeline Signed-off-by: Gustavo Carvalho <gustavo.carvalho@container-solutions.com> * fix: e2e tests with old docker buildx Signed-off-by: Gustavo Carvalho <gustavo.carvalho@container-solutions.com> * fix: coderabbit issues -- thanks coderabbit Signed-off-by: Gustavo Carvalho <gustavo.carvalho@container-solutions.com> * fix: force old bundle format for verify-attestation Signed-off-by: Gustavo Carvalho <gustavo.carvalho@container-solutions.com> * fix: typo Signed-off-by: Gustavo Carvalho <gustavo.carvalho@container-solutions.com> * fix: use-signing-config to false Signed-off-by: Gustavo Carvalho <gustavo.carvalho@container-solutions.com> * fix: verify attestation does not use signing config Signed-off-by: Gustavo Carvalho <gustavo.carvalho@container-solutions.com> * fix: readd arm64 for -ubi-boringssl Signed-off-by: Gustavo Carvalho <gustavo.carvalho@container-solutions.com> * fix: remove bundle format from verify attestation Signed-off-by: Gustavo Carvalho <gustavo.carvalho@container-solutions.com> * fix: cosign regression on 3.0.4 prevents old pipelines from working (despite flags) Signed-off-by: Gustavo Carvalho <gustavo.carvalho@container-solutions.com> * fix: ci bloat and provenance version Signed-off-by: Gustavo Carvalho <gustavo.carvalho@container-solutions.com> * fix: replace slsa-provenance-action with custom bash Signed-off-by: Gustavo Carvalho <gustavo.carvalho@container-solutions.com> --------- Signed-off-by: Gustavo Carvalho <gustavo.carvalho@container-solutions.com>



attempts to fix our image signature process
Summary
Fixes cosign compatibility issues in the image signature process by upgrading cosign from v3.0.3 to v3.0.4, forcing the older bundle format via --new-bundle-format=false and disabling signing-config via --use-signing-config=false for sign/attest/verify steps, relaxing the certificate identity regexp, and improving signing, SBOM, and provenance logging and verification flows.
Changes
.github/actions/sign/action.yml.github/workflows/helm.ymlhttps://github.com/$GITHUB_REPOSITORY/*tohttps://github.com/$GITHUB_REPOSITORY/.*for verification..github/actions/e2e/action.yml.github/workflows/e2e.ymlImpact