-
Notifications
You must be signed in to change notification settings - Fork 247
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
*: add kernel argument support #1178
Conversation
Okay, updated for comments. Additionally I went ahead and updated |
aa8839d
to
3aae119
Compare
⏫ Updated ⏫ |
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.
Can we add a test that configs will be rejected if the same value is in shouldExist
and shouldNotExist
?
Updated, will still potentially need another update pending questions in #1178 (comment) & #1178 (comment) |
Adds a new top-level struct kernelArguments which has two fields, shouldExist & shouldNotExist both of which accept lists of strings.
Adds a new stage kargs which will call out to an external binary to set kernel arguments during the first boot. This is done in a separate stage to allow the distribution to set kernel arguments before the disks stage is run and optionally reboot.
Updated |
ce69013
to
c4142c8
Compare
Adds a new kargs stage, as well as a binary implementation that prompts the user to contact their distribution to implement a kargs helper binary.
Related to coreos/ignition#1178 This change adds support for providing a kernelArgument field in the butane config.
Related to coreos/ignition#1178 This change adds support for providing a kernelArgument field in the butane config.
Related to coreos/ignition#1178 This change adds support for providing a kernelArgument field in the butane config.
Add a new top-level field
kernelArgument
to the configuration specification, a new Ignition stagekargs
, and distributor documentation.Closes #1168