Skip to content

feat(ISV-5859): Add index image SBOM generator #9

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

Merged
merged 1 commit into from
May 12, 2025
Merged

feat(ISV-5859): Add index image SBOM generator #9

merged 1 commit into from
May 12, 2025

Conversation

Allda
Copy link
Collaborator

@Allda Allda commented May 7, 2025

A Mobster can now generate SBOMs for index images that represents its multiarch structure. The script generates SBOMs in SPDX format using provided index manifest generated using Buildah/Skopeo.

The generator uses spdx_tools library to assemble the document and before storing it to file it validates its schema.

JIRA: ISV-5859

A Mobster can now generate SBOMs for index images that represents its
multiarch structure. The script generates SBOMs in SPDX format using
provided index manifest generated using Buildah/Skopeo.

The generator uses spdx_tools library to assemble the document and
before storing it to file it validates its schema.

JIRA: ISV-5859

Signed-off-by: Ales Raszka <[email protected]>
@Allda Allda requested review from mavaras and ezopezo May 7, 2025 07:43
@Allda
Copy link
Collaborator Author

Allda commented May 7, 2025

/retest


for manifest in index_manifest["manifests"]:
if manifest["mediaType"] not in self.IMAGE_MANIFEST_MEDIA_TYPES:
LOGGER.warning(
Copy link

Choose a reason for hiding this comment

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

I guess this is a case for potential emerging of new mediatype in future - shouldn't we rather end up with error and adapt script right away instead producing incomplete SBOM? Or are those warnings sufficiently visible in konflux runtime?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

We don't want to support any other mediatyes except those that are defined in the script. The warning is there tu just inform user that there is something that won't be present in the SBOM as it is not any convential image.

@Allda Allda merged commit 81f31fd into main May 12, 2025
4 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