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

Add support for syft json type to cosign #1137

Merged
merged 2 commits into from
Dec 6, 2021
Merged

Conversation

samj1912
Copy link
Contributor

@samj1912 samj1912 commented Dec 5, 2021

Signed-off-by: Sambhav Kothari [email protected]

Summary

Add support for syft format in cosign attach sbom

Ticket Link

Fixes #1136

Release Note

Add support for syft format in cosign attach sbom

@developer-guy
Copy link
Member

wow, thanks @samj1912, this is amazing! I do even know Syft but did not realize that they use their own SBOM format.

PTAL @luhring

@samj1912
Copy link
Contributor Author

samj1912 commented Dec 5, 2021

Not sure why the cosigned tests are failing but it looks unrelated to the PR

@developer-guy
Copy link
Member

Not sure why the cosigned tests are failing but it looks unrelated to the PR

yes, seems something is broken, this is not related to your PR 🤝

@dlorenc
Copy link
Member

dlorenc commented Dec 5, 2021

Looks great! Let's see if we can chase down the ci failure.

@dlorenc
Copy link
Member

dlorenc commented Dec 5, 2021

Looks like the distroless image is not actually signed: GoogleContainerTools/distroless#905

pkg/types/media.go Outdated Show resolved Hide resolved
@dlorenc dlorenc merged commit e48db5a into sigstore:main Dec 6, 2021
@github-actions github-actions bot added this to the v1.4.0 milestone Dec 6, 2021
@samj1912 samj1912 deleted the syft branch December 6, 2021 20:05
samj1912 added a commit to samj1912/cosign that referenced this pull request Jan 6, 2022
This documents the support for syft json added in sigstore#1137 

Signed-off-by: Sambhav Kothari <[email protected]>
dlorenc pushed a commit that referenced this pull request Jan 6, 2022
* Update SBOM spec to indicate compat for syft

This documents the support for syft json added in #1137 

Signed-off-by: Sambhav Kothari <[email protected]>

* Reword SBOM wording to indicate that the formats are cosign specific

As noted by @VinodAnandan - the previous message may have caused confusion about NTIA recorgnized formats v/s formats cosign uses. Updating the wording to explicitly call out cosign supported formats.

Signed-off-by: Sambhav Kothari <[email protected]>
mlieberman85 pushed a commit to mlieberman85/cosign that referenced this pull request May 6, 2022
* Update SBOM spec to indicate compat for syft

This documents the support for syft json added in sigstore#1137 

Signed-off-by: Sambhav Kothari <[email protected]>

* Reword SBOM wording to indicate that the formats are cosign specific

As noted by @VinodAnandan - the previous message may have caused confusion about NTIA recorgnized formats v/s formats cosign uses. Updating the wording to explicitly call out cosign supported formats.

Signed-off-by: Sambhav Kothari <[email protected]>
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.

Add support for syft SBOMs to cosign
4 participants