-
Notifications
You must be signed in to change notification settings - Fork 49
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
Generate the ALL-MANIFEST for bootc disk images #512
Conversation
We never used it in bootc-image-builder.
The bootc-disk image manifest now always contains all pipelines needed to generate all image types. This way we can build any combination of disk images depending on what we need to export without needing to conditionally add or remove pipelines. The Platform no longer needs to be read, since we're generating all of the formats and the architecture depends on the source container. However, the Platform is still required for specifying the boot mode (enabling or disabling BIOS boot). The first return value of the InstantiateManifestFromContainers() function, the artifact, was not used and has been removed now.
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.
This is amazing, THANK YOU!
(needs a linter fix)
990e432
to
20b398c
Compare
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.
Thank you!
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.
Neat!
The bootc-disk image manifest now always contains all pipelines needed to generate all image types. This way we can build any combination of disk images depending on what we need to export without needing to conditionally add or remove pipelines.
The Platform no longer needs to be read, since we're generating all of the formats and the architecture depends on the source container. However, the Platform is still required for specifying the boot mode (enabling or disabling BIOS boot).
The first return value of the InstantiateManifestFromContainers() function, the artifact, was not used and has been removed now.