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

Non-interactive anaconda ISO requires additional bootc switch --migrate-in-place #780

Open
p5 opened this issue Jan 6, 2025 · 4 comments
Assignees

Comments

@p5
Copy link

p5 commented Jan 6, 2025

When creating a BIB image without specifying a kickstart script in the configuration, the following command is added as a %post step:

bootc switch --mutate-in-place --transport %s %s

(source)

However, when the operator specifies their own kickstart (e.g. adds custom steps or an empty string to force an interactive installer), this step is not added through osbuild. Therefore, the operator must be aware of this command (which is not documented anywhere!) and explicitly add it to the config file.

This function which adds that bootc switch command seems to be exiting early when the operator provides any sort of content in the kickstart (including a blank string - ""), so the command wouldn't be present in the resulting ISO unless this is explicitly defined in the config.toml input.

@supakeen supakeen self-assigned this Jan 6, 2025
@supakeen
Copy link
Member

supakeen commented Jan 6, 2025

I feel that since we have ostreecontainer in our kickstart unconditionally (even when a user provided kickstart is there), that we should probably do the same with bootc switch?

Alternatively we can document that users need to add this always but that seems a bit less friendly.

cc @mvo5 / @achilleas-k.

@cgwalters
Copy link
Contributor

Yes, we should add it automatically.

@supakeen
Copy link
Member

supakeen commented Jan 7, 2025

Yes, we should add it automatically.

Before I make the change in images, is it intended to be permanent or is the plan that kickstarts' ostreecontainer be changed to take care of this eventually?

@ondrejbudai
Copy link
Member

The proper fix is anaconda getting a proper support for bootc, see: rhinstaller/anaconda#5197

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

No branches or pull requests

4 participants