Skip to content

Add a generic container workflow#471

Merged
ianlewis merged 33 commits intoslsa-framework:mainfrom
ianlewis:409-feature-add-generic-container-workflow
Jul 14, 2022
Merged

Add a generic container workflow#471
ianlewis merged 33 commits intoslsa-framework:mainfrom
ianlewis:409-feature-add-generic-container-workflow

Conversation

@ianlewis
Copy link
Member

@ianlewis ianlewis commented Jul 1, 2022

Updates #409

Adds a "generic" container workflow. Users build the container in the user workflow and call the reusable container workflow which generates provenance using cosign attest and uploads it to the container registry.

cosign attest uploads additional metadata to the registry which is used during the verification process by cosign verify-attestation. The generic workflow cannot support uploading this additional metadata because our generic workflow is not container aware and users cannot upload the attestation produced by the workflow with cosign attach attestation because it does not upload it.

This PR adds a new workflow that is container aware. Users build and push their image to a registry. This workflow will use the generic generator to produce a SLSA predicate and then run the cosign attest command to sign and upload the provenance.

Support for slsa-verifier is being implemented in slsa-framework/slsa-verifier#92

Depends on:

@ianlewis ianlewis mentioned this pull request Jul 5, 2022
7 tasks
@ianlewis ianlewis marked this pull request as ready for review July 6, 2022 01:47
Copy link
Member

@joshuagl joshuagl left a comment

Choose a reason for hiding this comment

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

This looks great, thanks

@ianlewis ianlewis merged commit aacb56f into slsa-framework:main Jul 14, 2022
@ianlewis ianlewis deleted the 409-feature-add-generic-container-workflow branch July 27, 2022 01:21
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.

3 participants