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
boot: load bits of kernel command line from gadget snaps #10130
boot: load bits of kernel command line from gadget snaps #10130
Conversation
Load the kernel command line arguments provided by the gadget if the path to a gadget snap directory or file is provided. Signed-off-by: Maciej Borzecki <maciej.zenon.borzecki@canonical.com>
…t-load-cmdline-from-gadget
…t-load-cmdline-from-gadget
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.
thanks, one remark
extraArgs := "" | ||
if gadgetDirOrSnapPath != "" { | ||
sf, err := snapfile.Open(gadgetDirOrSnapPath) |
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.
shouldn't we merge this Open into gadget.KernelCommandLineFromGadget ? do we have a case where we have the container already?
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.
Yes, that's a good point. I'll do it in a followup so that #10134 is unblocked first, if that's ok with you.
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.
Thank you
…t-load-cmdline-from-gadget
All failures look unreleated |
Stacked on top of #10119, the relevant commit is: af2862f
The branch adds support for loading bits of kernel command line from the gadget. Currently only cmdline.extra is supported.