Skip to content

ci: reference slsa-github-generator by tag (fix exit 127 / missing provenance)#6

Merged
YoungDan merged 1 commit into
mainfrom
ci/slsa-tag-pin
May 29, 2026
Merged

ci: reference slsa-github-generator by tag (fix exit 127 / missing provenance)#6
YoungDan merged 1 commit into
mainfrom
ci/slsa-tag-pin

Conversation

@YoungDan
Copy link
Copy Markdown
Member

Summary

  • Change the SLSA reusable workflow reference from a commit SHA to the @v2.1.0 tag.

Why

Both v0.1.0-rc.1 and v0.1.0-rc.2 dry-runs failed the SLSA jobs identically. Root cause: the slsa-github-generator reusable workflow downloads its builder binary from the release matching its own ref. When pinned by a 40-char SHA there is no matching release, the binary is never fetched, and the attest step dies with exit 127 (slsa-generator-generic-linux-amd64: No such file or directory) — leaving an empty provenance that fails upload-assets and final.

Referencing by the semver tag is the project's documented requirement; the generator self-verifies the downloaded builder against the release. A code comment records this as an explicit, justified exception to the repo's SHA-pinning policy.

Test plan

  • CI green.
  • After merge, tag v0.1.0-rc.3 → all SLSA jobs (generator, upload-assets, final) pass and multiple.intoto.jsonl is attached to the prerelease.

The slsa-github-generator reusable workflow downloads its builder binary
from the release matching its own ref. SHA-pinning it means there is no
matching release to resolve, so the binary is never fetched and the
attest step dies with exit 127 (No such file or directory), leaving an
empty provenance that fails the upload-assets and final jobs.

Pin by the v2.1.0 tag (documented requirement); a comment records why
this is an explicit exception to the SHA-pinning policy.
@YoungDan YoungDan merged commit 88becb5 into main May 29, 2026
4 checks passed
@YoungDan YoungDan deleted the ci/slsa-tag-pin branch May 29, 2026 11:42
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.

1 participant