Skip to content

build/bake: fail fast on dependency signature verification errors#252

Merged
crazy-max merged 1 commit into
docker:mainfrom
crazy-max:fix-dependency-signature-verification
Jul 18, 2026
Merged

build/bake: fail fast on dependency signature verification errors#252
crazy-max merged 1 commit into
docker:mainfrom
crazy-max:fix-dependency-signature-verification

Conversation

@crazy-max

@crazy-max crazy-max commented Jul 14, 2026

Copy link
Copy Markdown
Member

relates to https://github.com/crazy-max/xgo/actions/runs/29336003773/job/87096835413?pr=198#step:4:1

image image

Dependency signature verification now reports failures after the log group closes so the error is visible in the workflow step summary.

The verification loop now rethrows the first failure instead of only marking the action as failed, which stops later dependency checks from running after an invalid signature result.

Also looking at the error:

Error: Error: Cosign verify command failed with: error during command execution: trusted root is required when using new bundle format

It seems cosign is not able to download tuf (probably flaky network). This could be solved with docker/actions-toolkit#1133.

@crazy-max
crazy-max requested a review from tonistiigi July 14, 2026 13:45
@crazy-max
crazy-max requested a review from a team as a code owner July 14, 2026 13:45
@crazy-max
crazy-max force-pushed the fix-dependency-signature-verification branch 2 times, most recently from 1a53399 to d7d0dcd Compare July 15, 2026 08:52
@crazy-max

Copy link
Copy Markdown
Member Author

Comment thread .github/workflows/bake.yml Outdated
});
} catch (error) {
core.setFailed(`Failed to verify ${image}: ${error.message || error}`);
break;

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.

nit: this could just be return?

Signed-off-by: CrazyMax <1951866+crazy-max@users.noreply.github.com>
@crazy-max
crazy-max force-pushed the fix-dependency-signature-verification branch from d7d0dcd to 892d4f4 Compare July 18, 2026 14:39
@crazy-max
crazy-max merged commit 288062a into docker:main Jul 18, 2026
171 checks passed
@crazy-max
crazy-max deleted the fix-dependency-signature-verification branch July 18, 2026 14:41
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