Merge pull request coreos#433 from cgwalters/install-gpt

bios: Always add `part_gpt`
cgwalters · Feb 28, 2023 · eba79f3 · eba79f3
2 parents f187052 + c4c58ef
commit eba79f3
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/src/bios.rs b/src/bios.rs
@@ -58,10 +58,13 @@ impl Bios {
 
         let mut cmd = Command::new(grub_install);
         let boot_dir = Path::new(dest_root).join("boot");
+        // We forcibly inject mdraid1x because it's needed by CoreOS's default of "install raw disk image"
+        // We also add part_gpt because in some cases probing of the partition map can fail such
+        // as in a container, but we always use GPT.
         #[cfg(target_arch = "x86_64")]
         cmd.args(&["--target", "i386-pc"])
             .args(&["--boot-directory", boot_dir.to_str().unwrap()])
-            .args(&["--modules", "mdraid1x"])
+            .args(&["--modules", "mdraid1x part_gpt"])
             .arg(device);
 
         #[cfg(target_arch = "powerpc64")]