diff mbox series

[isar-cip-core] Revert "Removed "ro" option from read-only-rootfs's fstab."

Message ID 3807cd22-d5d5-4cac-88ab-09643573c527@siemens.com (mailing list archive)
State New
Headers show
Series [isar-cip-core] Revert "Removed "ro" option from read-only-rootfs's fstab." | expand

Commit Message

Jan Kiszka Feb. 24, 2025, 1:25 p.m. UTC 
From: Jan Kiszka <jan.kiszka@siemens.com>

This reverts commit a36d28e946e6edb7fb27883e6ecb398001269f92.

It turned out that systemd-remount-fs dislikes it when "defaults" or
even spelled-out "defaults" without rw suggests it should mount /
writable. Result is a failing service and degraded boot.

If some future use case actually needs rw due to adding some overlay for
root again, that will have to address the issue differently.

Signed-off-by: Jan Kiszka <jan.kiszka@siemens.com>
---
 classes/read-only-rootfs.bbclass | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
diff mbox series

Patch

diff --git a/classes/read-only-rootfs.bbclass b/classes/read-only-rootfs.bbclass
index 83ddc336..35a3ab3d 100644
--- a/classes/read-only-rootfs.bbclass
+++ b/classes/read-only-rootfs.bbclass
@@ -41,7 +41,7 @@  SQUASHFS_EXCLUDE_DIRS = "${RO_ROOTFS_EXCLUDE_DIRS}"
 image_configure_fstab() {
     sudo tee '${IMAGE_ROOTFS}/etc/fstab' << EOF
 # Begin /etc/fstab
-/dev/root	/		auto		defaults			0	0
+/dev/root	/		auto		defaults,ro			0	0
 LABEL=var	/var		auto		defaults			0	0
 proc		/proc		proc		nosuid,noexec,nodev		0	0
 sysfs		/sys		sysfs		nosuid,noexec,nodev		0	0