-
Notifications
You must be signed in to change notification settings - Fork 547
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 cosign download sbom when image is absent #3245
Conversation
Signed-off-by: Vishal Choudhary <[email protected]>
Codecov Report
@@ Coverage Diff @@
## main #3245 +/- ##
==========================================
+ Coverage 30.35% 30.93% +0.58%
==========================================
Files 155 155
Lines 9845 9929 +84
==========================================
+ Hits 2988 3072 +84
+ Misses 6410 6404 -6
- Partials 447 453 +6 |
Signed-off-by: Vishal Choudhary <[email protected]>
Signed-off-by: Vishal Choudhary <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Seems good to me, but I'm not familiar with the code path - @imjasonh @hectorj2f can you confirm this LGTY?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, although we could add comments in the code to explain it.
Signed-off-by: Vishal Choudhary <[email protected]>
6995070
@haydentherapper @hectorj2f Added comments can you please review? |
Summary
Closes: #2603
Cosign will not fail to download sbom when image is absent.
This PR build on top of changes made in: #2959
Proof:
Release Note
Added support for
cosign download sbom
when image is absentDocumentation