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 parent images for containers #23

Merged
merged 6 commits into from
Oct 7, 2024
Merged

Add parent images for containers #23

merged 6 commits into from
Oct 7, 2024

Conversation

twaugh
Copy link
Collaborator

@twaugh twaugh commented Sep 10, 2024

Use BUILD_TOOL_OF for multi-stage builds, and DESCENDANT_OF for direct ancestors.

@twaugh twaugh requested a review from mprpic September 10, 2024 10:50
@twaugh
Copy link
Collaborator Author

twaugh commented Sep 10, 2024

This is a first approximation. Still missing: checksums for the parent images, which could be retrieved from Koji.

@twaugh twaugh force-pushed the container-parent-images branch from 52005ca to 6bf3c6f Compare September 10, 2024 11:05
@twaugh twaugh force-pushed the container-parent-images branch 2 times, most recently from 20ed291 to eb1d988 Compare September 11, 2024 12:56
@twaugh twaugh force-pushed the container-parent-images branch from eb1d988 to 9019fba Compare September 12, 2024 10:26
@mprpic
Copy link
Contributor

mprpic commented Sep 12, 2024

@twaugh Are we able to pull out the information about the Go components used in the build? Right now the kmm SBOM contains just RPMs.

@mprpic
Copy link
Contributor

mprpic commented Sep 25, 2024

@twaugh Can this be merged or is it waiting on anything else?

@twaugh twaugh force-pushed the container-parent-images branch from 9019fba to 065db62 Compare October 7, 2024 15:28
@twaugh twaugh merged commit 97c6339 into main Oct 7, 2024
3 checks passed
@twaugh twaugh deleted the container-parent-images branch October 7, 2024 15:51
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.

2 participants