Merge pull request coreos#134 from jlebon/pr/mount-ro

30ignition/ignition-setup-user: mount /boot partition ro
cgwalters · Nov 7, 2019 · d9e8a63 · d9e8a63
2 parents 38af701 + e96e7fc
commit d9e8a63
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/dracut/30ignition/ignition-setup-user.sh b/dracut/30ignition/ignition-setup-user.sh
@@ -25,6 +25,8 @@ else
     # with MountFlags=slave so it is unmounted for us.
     bootmnt=/mnt/boot_partition
     mkdir -p $bootmnt
-    mount /dev/disk/by-label/boot $bootmnt
+    # mount as read-only since we don't strictly need write access and we may be
+    # running alongside other code that also has it mounted ro
+    mount -o ro /dev/disk/by-label/boot $bootmnt
     copy_file_if_exists "${bootmnt}/ignition/config.ign" "${destination}/user.ign"
 fi