Skip to content

ci: skip windows release checksums and sigstore bundles - #3978

Merged
crazy-max merged 1 commit into
docker:masterfrom
crazy-max:ci-skip-sigstore-windows
Jul 28, 2026
Merged

ci: skip windows release checksums and sigstore bundles#3978
crazy-max merged 1 commit into
docker:masterfrom
crazy-max:ci-skip-sigstore-windows

Conversation

@crazy-max

Copy link
Copy Markdown
Member

This updates the Buildx release workflow so Windows artifacts are handled like Darwin artifacts before the external signing handoff.

Windows binaries are signed by a downstream workflow, which downloads the Buildx release assets, keeps the Darwin and Windows binaries, signs them, verifies the signatures, and uploads the signed binaries back to the release. Skipping Windows checksums here avoids publishing checksums for pre-signed binaries that will be replaced by the signing workflow.

Signed-off-by: CrazyMax <1951866+crazy-max@users.noreply.github.com>
mv "sbom-binaries.spdx.json" "${filename}.sbom.json"
find . -name 'sbom*.json' -exec rm {} \;
if [[ "$binname" == *darwin* ]]; then
if [[ "$binname" == *darwin* ]] || [[ "$binname" == *windows* ]]; then

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.

Do we have tracking issues for the remaining work for this? (wondering if a TODO + link is warranted here)

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

No public tracking issue that I know of yet. The intent here is to match the existing Darwin flow because Windows binaries will now be signed by the downstream workflow after the Buildx release assets are uploaded, so we should not publish checksums for the unsigned Windows binaries that will be replaced.

I will look to add something in our README.

@thaJeztah thaJeztah left a comment

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.

LGTM

left a suggestion

@crazy-max
crazy-max merged commit 4140767 into docker:master Jul 28, 2026
161 checks passed
@crazy-max
crazy-max deleted the ci-skip-sigstore-windows branch July 28, 2026 10:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants