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

fix: allow bom build and verification for build_only layers #610

Merged
merged 1 commit into from
May 10, 2024

Conversation

rchincha
Copy link
Contributor

@rchincha rchincha commented Apr 4, 2024

fix: allow bom build and verification for build_only layers

stacker builds allow chaining of layer builds.  SBOM chaining should
follow that model, so bom generation directives are concerned with only
that layer.

What type of PR is this?

Which issue does this PR fix:

What does this PR do / Why do we need it:

If an issue # is not available please add repro steps and logs showing the issue:

Testing done on this change:

Automation added to e2e:

Will this break upgrades or downgrades?

Does this PR introduce any user-facing change?:


By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@rchincha rchincha requested review from smoser and hallyn as code owners April 4, 2024 22:57
@rchincha rchincha force-pushed the bom branch 2 times, most recently from b887686 to e3a22fe Compare April 5, 2024 04:20
Copy link

codecov bot commented Apr 5, 2024

Codecov Report

Attention: Patch coverage is 50.00000% with 6 lines in your changes are missing coverage. Please review.

Project coverage is 57.35%. Comparing base (25b859b) to head (b78861e).
Report is 1 commits behind head on main.

❗ Current head b78861e differs from pull request most recent head 9cc8326. Consider uploading reports for the commit 9cc8326 to get more accurate results

Files Patch % Lines
pkg/overlay/pack.go 45.45% 5 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #610      +/-   ##
==========================================
+ Coverage   55.12%   57.35%   +2.23%     
==========================================
  Files          65       65              
  Lines        7715     7718       +3     
==========================================
+ Hits         4253     4427     +174     
+ Misses       2701     2534     -167     
+ Partials      761      757       -4     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@rchincha rchincha force-pushed the bom branch 2 times, most recently from 2cb4c1b to 10d7c60 Compare April 25, 2024 19:54
@rchincha rchincha changed the title fix: compress overlay_dir import fix: allow bom build and verification for build_only layers Apr 25, 2024
@rchincha rchincha force-pushed the bom branch 2 times, most recently from 88c2048 to 7de57a8 Compare May 8, 2024 18:27
stacker builds allow chaining of layer builds.  SBOM chaining should
follow that model, so bom generation directives are concerned with only
that layer.

Signed-off-by: Ramkumar Chinchani <[email protected]>
@rchincha rchincha merged commit 98ca66e into project-stacker:main May 10, 2024
10 checks passed
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