Skip to content

main: error when image-builder is used to create anaconda-iso#401

Merged
mvo5 merged 1 commit intoosbuild:mainfrom
mvo5:error-on-anaconda-iso
Dec 12, 2025
Merged

main: error when image-builder is used to create anaconda-iso#401
mvo5 merged 1 commit intoosbuild:mainfrom
mvo5:error-on-anaconda-iso

Conversation

@mvo5
Copy link
Contributor

@mvo5 mvo5 commented Dec 11, 2025

The behavior of anaconda-iso without specil mTLS setup is different from bib so instead of introducing subtle incompatibilities just error.

Note that we could support this with something like: https://github.com/osbuild/image-builder-cli/compare/main...mvo5:consolidate-bib-anaconda-iso-wip?expand=1 but the general code that is currently fairly nice becomes not so nice anymore as we need to add a bunch of extra things like extracting mTLS, cleanup etc for an image-type we consider legacy and want to replace with the bootc-installer image type.

@mvo5 mvo5 requested a review from a team as a code owner December 11, 2025 13:43
@mvo5 mvo5 requested review from croissanne, lzap and thozza and removed request for a team December 11, 2025 13:43
thozza
thozza previously approved these changes Dec 11, 2025
The behavior of anaconda-iso without specil mTLS setup is different
from bib so instead of introducing subtle incompatibilities just
error.

Note that we could support this with something like:
https://github.com/osbuild/image-builder-cli/compare/main...mvo5:consolidate-bib-anaconda-iso-wip?expand=1
but the general code that is currently fairly nice becomes not
so nice anymore as we need to add a bunch of extra things like
extracting mTLS, cleanup etc for an image-type we consider legacy
and want to replace with the `bootc-installer` image type.
Copy link
Contributor

@bcl bcl left a comment

Choose a reason for hiding this comment

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

Looks good!

Copy link
Contributor

@lzap lzap left a comment

Choose a reason for hiding this comment

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

I mean, sure.

@mvo5 mvo5 added this pull request to the merge queue Dec 12, 2025
@mvo5 mvo5 requested a review from achilleas-k December 12, 2025 11:22
Merged via the queue into osbuild:main with commit 27ef2b0 Dec 12, 2025
39 checks passed
@mvo5 mvo5 deleted the error-on-anaconda-iso branch December 12, 2025 11:44
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.

4 participants