Skip to content

Do not include the root= dracut config in PXE images#2377

Merged
teclator merged 2 commits intomasterfrom
remove_root_config_PXE
May 21, 2025
Merged

Do not include the root= dracut config in PXE images#2377
teclator merged 2 commits intomasterfrom
remove_root_config_PXE

Conversation

@teclator
Copy link
Contributor

Problem

The unconditional set of root= setting in the initrd is problematic in case of PXE images.

From the bug report:

.... while this might work for live ISO's booting via the dracut live (livenet) module it is a wrong and conflicting setting for the kiwi-dump module code which is active if you are not building a live image but an oem image with installpxe="true".

The unconditional setting for root=live:... causes the dracut live module code to be called and not the actual expected kiwi-dump code. When mixing the kernel cmdline parameters for both modules there is also a timing issue on the probing of the network such that sometimes one or the other module wins.

In short this unconditional root= setting causes a lot of issues and should only be done in the live ISO builds of agama and nowhere else.

Solution

Remove the root= configuration for PXE images.

Copy link

@mtomaschewski mtomaschewski left a comment

Choose a reason for hiding this comment

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

Looks fine to me.

@teclator teclator requested review from imobachgs and lslezak May 21, 2025 12:50
@teclator teclator merged commit 3e8521d into master May 21, 2025
8 checks passed
@teclator teclator deleted the remove_root_config_PXE branch May 21, 2025 13:07
teclator added a commit that referenced this pull request May 21, 2025
Fix for #2377 as it does the contrary, it sets the root= configuration
in PXE images.
@imobachgs imobachgs mentioned this pull request May 26, 2025
imobachgs added a commit that referenced this pull request May 26, 2025
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