Skip to content

Stack BOM RFC#125

Closed
sophiewigmore wants to merge 5 commits intomainfrom
stack-syft-bom
Closed

Stack BOM RFC#125
sophiewigmore wants to merge 5 commits intomainfrom
stack-syft-bom

Conversation

@sophiewigmore
Copy link
Copy Markdown
Member

@sophiewigmore sophiewigmore commented Nov 8, 2021

Summary

Readable

Use Cases

Checklist

  • I have viewed, signed, and submitted the Contributor License Agreement.
  • I have linked issue(s) that this PR should close using keywords or the Github UI (See docs)
  • I have added an integration test, if necessary.
  • I have reviewed the styleguide for guidance on my code quality.
  • I'm happy with the commit history on this PR (I have rebased/squashed as needed).

@sophiewigmore sophiewigmore marked this pull request as ready for review November 8, 2021 22:25
@sophiewigmore sophiewigmore requested a review from a team as a code owner November 8, 2021 22:25
Comment thread text/stacks/0003-sbom-formats.md Outdated
Comment thread text/stacks/0003-sbom-formats.md Outdated
Comment thread text/stacks/0003-sbom-formats.md Outdated
Sophie Wigmore and others added 3 commits November 8, 2021 18:39
Co-authored-by: Ryan Moran <155736+ryanmoran@users.noreply.github.com>
Co-authored-by: Ryan Moran <155736+ryanmoran@users.noreply.github.com>
Co-authored-by: Ryan Moran <155736+ryanmoran@users.noreply.github.com>
Copy link
Copy Markdown

@brayanhenao brayanhenao left a comment

Choose a reason for hiding this comment

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

Looks good to me

@dmikusa
Copy link
Copy Markdown
Contributor

dmikusa commented Nov 9, 2021

Same. +1 looks good.

@dmikusa dmikusa self-requested a review November 9, 2021 16:19
@ryanmoran
Copy link
Copy Markdown
Contributor

This looks great! We should block on the upstream RFC getting merged though so that we can track any changes between now and then.

sophiewigmore pushed a commit to paketo-buildpacks/stacks that referenced this pull request Nov 19, 2021
- Implements paketo-buildpacks/rfcs#125
- Uses layer addition code from github.com/sclevine/cnb-sbom
- Includes basic BOM generation and attachment logic
sophiewigmore pushed a commit to paketo-buildpacks/stacks that referenced this pull request Nov 19, 2021
- Implements paketo-buildpacks/rfcs#125
- Uses layer addition code from github.com/sclevine/cnb-sbom
- Includes basic BOM generation and attachment logic
sophiewigmore pushed a commit to paketo-buildpacks/stacks that referenced this pull request Nov 29, 2021
- Although not consistent with github.com/paketo-buildpacks/rfcs/pull/125, it is consistent with
  comments in the upstream CNB, and can be easily removed.
sophiewigmore pushed a commit to paketo-buildpacks/stacks that referenced this pull request Nov 29, 2021
- Although not consistent with github.com/paketo-buildpacks/rfcs/pull/125, it is consistent with
  comments in the upstream CNB, and can be easily removed.
brayanhenao pushed a commit to paketo-buildpacks/stacks that referenced this pull request Dec 9, 2021
* Add run image BOM

- Implements paketo-buildpacks/rfcs#125
- Uses layer addition code from github.com/sclevine/cnb-sbom
- Includes basic BOM generation and attachment logic

* use specific alpine tag in bom generation test

* Change bom file name to include original image digest

- Although not consistent with github.com/paketo-buildpacks/rfcs/pull/125, it is consistent with
  comments in the upstream CNB, and can be easily removed.

* generate Squashed Syft output

* Attach SBOM using feature flag

* Assert SBOM doesn't appear when not flagged

* Adds experimental SBOM support in create-stack action

Co-authored-by: Ryan Moran <rmoran@vmware.com>
Co-authored-by: Sophie Wigmore <swigmore@vmware.com>
@sophiewigmore
Copy link
Copy Markdown
Member Author

Closing out for now due to inactivity. Will re-open or create a new RFC when the direction upstream is definitive.

@ryanmoran ryanmoran deleted the stack-syft-bom branch July 18, 2022 16:02
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.

4 participants