-
Notifications
You must be signed in to change notification settings - Fork 305
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
Provide a way to override the ostree= kernel argument #1920
Comments
Hmm, I wonder if we could just fallback to comparing |
Discussions about this in #1995. |
I don't understand, it's ostree-prepare-root that does the |
Yeah, let me unpack that. The idea with that suggestion is to just not teach libostree at all about live systems, but instead make it more generically about "no bootloader" mode signalled by having no
|
I think we're on the same page, but how about instead of:
We require that OS builders create a symlink:
in the physical root. That way ostree-prepare-root doesn't have to do any filesystem traversal, and it's a much more explicit notice of intent. |
I think I agree, and we may just be using words differently. When you say "read-only", you're talking about read-only bootloader configuration, right? Because otherwise, I don't see why most operations that normally require a writable sysroot shouldn't keep working in a live system. IOW, we would error out on |
Right. Hmm.
Oh definitely...and actually there's no reason to tie that to "live/no-/boot systems", see also coreos/fedora-coreos-tracker#354 (comment) |
ostree-prepare-root
andostree-system-generator
both read the deploy path solely from/proc/cmdline
. On live PXE systems, the deploy path doesn't come from kernel args; instead, the initramfs synthesizes it by inspecting the filesystem. At present, the only way to pass this path to ostree is to write a fake kernel command line to a file and bind-mount it on top of/proc/cmdline
.It would be helpful for ostree to provide a way to override the deploy path, or perhaps override its view of the kernel command line. This might be an environment variable or command-line option.
The text was updated successfully, but these errors were encountered: