-
Notifications
You must be signed in to change notification settings - Fork 28
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
bios+ppc64le: Support being passed a whole device #839
Conversation
a92a4cc
to
da76293
Compare
This depends on containers/bootc#1086 |
OK I only compile tested this locally but if someone has a reproducer env, can you try e.g.:
|
aede886
to
8e1bb34
Compare
Do testing on ppc64, failed to run update, here are the steps. Will look at it more tomorrow.
|
82ae251
to
f89818e
Compare
OK yeah digging in we're hitting udev issues; We'll need to switch to |
f89818e
to
9175a0a
Compare
On general principle but this is also prep for coreos#839
And automatically find the PReP partition. Closes: coreos#837 Signed-off-by: Colin Walters <[email protected]>
9175a0a
to
63af2d0
Compare
This will depend on coreos/bootupd#839 And cleans up our bootloader code; move the bootloader partition definitions back to `install/baseline`. Conceptually now it's just the `to-disk` flow and bootupd that know about bootloader partition GUIDs which is how I think it should be - the bootc core is agnostic to bootloaders. Signed-off-by: Colin Walters <[email protected]>
OK, I've tested this one now in concert with containers/bootc#1106 and it looks like it works. I did hit an issue when trying to use Ideally we figure out a generic way to probe this in containers/bootc#192
I think I hadn't updated that binary with the latest from this PR before, but I have now. Want to retest? |
I understood the problem now, can reproduce using
Sorry that it is introduced by 07c3661 which ignore the ppc64le platform. Retest and run |
@@ -12,6 +14,30 @@ use crate::packagesystem; | |||
// grub2-install file path | |||
pub(crate) const GRUB_BIN: &str = "usr/sbin/grub2-install"; | |||
|
|||
#[cfg(target_arch = "powerpc64")] | |||
fn target_device(device: &str) -> Result<Cow<str>> { |
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.
Just one minor suggestion, not blocker, better to rename target_device
to get_prepboot_device()
, and move it to src/blockdev.rs
? I am also OK to keep it here as it is only used on ppc64.
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.
LGTM
And automatically find the PReP partition.
Closes: #837