Skip to content

Add an OperatingSystem package to our image SBOMs#1690

Merged
sil2100 merged 4 commits into
chainguard-dev:mainfrom
sil2100:sbom-os-info
May 29, 2025
Merged

Add an OperatingSystem package to our image SBOMs#1690
sil2100 merged 4 commits into
chainguard-dev:mainfrom
sil2100:sbom-os-info

Conversation

@sil2100
Copy link
Copy Markdown
Member

@sil2100 sil2100 commented May 28, 2025

We are missing some information in our image SBOMs that cause trivy to generate warnings. The missing piece is a package declaring what operating system our images are using.

sil2100 added 2 commits May 28, 2025 18:31
Signed-off-by: Łukasz 'sil2100' Zemczak <lukasz.zemczak@chainguard.dev>
Signed-off-by: Łukasz 'sil2100' Zemczak <lukasz.zemczak@chainguard.dev>
@sil2100 sil2100 marked this pull request as ready for review May 28, 2025 16:54
Comment thread pkg/sbom/generator/spdx/spdx.go Outdated
Comment thread pkg/sbom/generator/spdx/spdx.go Outdated
Signed-off-by: Łukasz 'sil2100' Zemczak <lukasz.zemczak@chainguard.dev>
xnox
xnox previously requested changes May 28, 2025
Comment thread pkg/sbom/generator/spdx/spdx.go
Signed-off-by: Łukasz 'sil2100' Zemczak <lukasz.zemczak@chainguard.dev>
@sil2100
Copy link
Copy Markdown
Member Author

sil2100 commented May 28, 2025

Ok, remaining failures:

  • build-all-examples seems to be unrelated to this change - looks like apk issues?
  • build-nginx-all-arches seems to be the - and _ thing

@sil2100 sil2100 enabled auto-merge (squash) May 29, 2025 08:09
@sil2100
Copy link
Copy Markdown
Member Author

sil2100 commented May 29, 2025

Tested against a locally built image: the sbom seems to be correct and trivy seems to be happy with it. Ship it!

Copy link
Copy Markdown
Contributor

@javacruft javacruft left a comment

Choose a reason for hiding this comment

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

Previous discussion on this PR is now resolved so 👍

@sil2100 sil2100 dismissed xnox’s stale review May 29, 2025 09:09

All mentioned issues were valid and have been resolved. Thanks!

@sil2100 sil2100 merged commit 914a574 into chainguard-dev:main May 29, 2025
16 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.

3 participants