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

Handle kernel arguments better, especially when duplicated #604

Open
rdoxenham opened this issue Nov 6, 2024 · 0 comments · May be fixed by #605
Open

Handle kernel arguments better, especially when duplicated #604

rdoxenham opened this issue Nov 6, 2024 · 0 comments · May be fixed by #605
Labels
enhancement New feature or request triaged The team has reviewed the issue
Milestone

Comments

@rdoxenham
Copy link
Contributor

Earlier this week, an issue was raised in nmc (suse-edge/nm-configurator#122) related to the way that EIB is handling and configuring kernel arguments, and how in this specific case, net.ifnames=1 wasn't being honoured due to the way that SL Micro 6.0 kexec's into the new disk image post image dump, and uses a pre-configured kernel cmdline set at Kiwi image build time.

In short, we end up in a situation where a user can specify:

operatingSystem:
  kernelArgs:
    - net.ifnames=1

Yet the resulting kernel cmdline after kexec adds in the Kiwi-set ones, which by default includes net.ifnames=0, and thus ignores the user-provided argument. Earlier in the year, @e-minguez raised #369 which should also be considered where we simply rely on the ordering of the kernel cmdline arguments, but we should handle this better.

@rdoxenham rdoxenham linked a pull request Nov 6, 2024 that will close this issue
@jdob jdob added this to the v1.2 milestone Nov 22, 2024
@jdob jdob added enhancement New feature or request triaged The team has reviewed the issue labels Nov 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request triaged The team has reviewed the issue
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants