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

build-push-to-dockerhub: Generate SBOM & attestations #120

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

iainlane
Copy link
Member

@iainlane iainlane commented May 8, 2024

This generates an SBOM and attestations for that SBOM and also build provenance. These are also stored in one of GitHub's SigStore instances, so the provenance of images built using this action can be verified.

We do this by repurposing the syft-sbom-report action and calling that as a nested composite action, to encapsulate the generation and pushing work. This wasn't a usable action (no action.yaml) anyway. It was already referencing the anchore/sbom-action we use here.

@iainlane iainlane force-pushed the iainlane/attestations branch 2 times, most recently from c39e4b2 to d51a70c Compare May 13, 2024 14:49
This generates an SBOM and provenance attestations. These are also
stored in one of GitHub's SigStore instances, so the provenance of
images built using this action can be verified.

We delete the `syft-sbom-report` action. This wasn't a usable action (no
`action.yaml`) anyway.

What we do here is based on [a comment][comment] from one of the GitHub
attestation folks. We only generate the provenance attestation with
GitHub directly, and use Docker to generate the SBOM and another
provenance attestation about the Dockerish bits of the build. The GitHub
attestation acts as a signature over all of that, and should handle
multi-arch manifests properly, which isn't possible with the documented
GitHub way that exists currently.

[comment]: actions/attest-sbom#60 (comment)
@iainlane iainlane force-pushed the iainlane/attestations branch 3 times, most recently from 396c261 to e92e02e Compare July 26, 2024 07:18
Copy link
Contributor

Automated rebase attempt failed. Please rebase manually.

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