]> git.ipfire.org Git - thirdparty/mkosi.git/commitdiff
Move mount options into original declaration. 1844/head
authorBlair Bonnett <blair.bonnett@gmail.com>
Tue, 29 Aug 2023 13:22:19 +0000 (15:22 +0200)
committerBlair Bonnett <blair.bonnett@gmail.com>
Tue, 29 Aug 2023 13:22:19 +0000 (15:22 +0200)
mkosi/mounts.py

index a026a41dac48088c722cab1c5beceefb798e1d3b..0a5e5fc4a57df7946d3053894d32cb61603f2959 100644 (file)
@@ -81,16 +81,19 @@ def mount_overlay(
     read_only: bool = True,
 ) -> Iterator[Path]:
     with tempfile.TemporaryDirectory(dir=upperdir.parent, prefix=f"{upperdir.name}-workdir") as workdir:
-        options = [f"lowerdir={lower}" for lower in lowerdirs] + [f"upperdir={upperdir}", f"workdir={workdir}"]
-
-        # Disable the inodes index and metacopy (only copy metadata upwards if possible)
-        # options. If these are enabled (e.g., if the kernel enables them by default),
-        # the mount will fail if the upper directory has been earlier used with a different
-        # lower directory, such as with a build overlay that was generated on top of a
-        # different temporary root.
-        # See https://www.kernel.org/doc/html/latest/filesystems/overlayfs.html#sharing-and-copying-layers
-        # and https://github.com/systemd/mkosi/issues/1841.
-        options.extend(["index=off", "metacopy=off"])
+        options = [f"lowerdir={lower}" for lower in lowerdirs] + [
+            f"upperdir={upperdir}",
+            f"workdir={workdir}",
+            # Disable the inodes index and metacopy (only copy metadata upwards if possible)
+            # options. If these are enabled (e.g., if the kernel enables them by default),
+            # the mount will fail if the upper directory has been earlier used with a different
+            # lower directory, such as with a build overlay that was generated on top of a
+            # different temporary root.
+            # See https://www.kernel.org/doc/html/latest/filesystems/overlayfs.html#sharing-and-copying-layers
+            # and https://github.com/systemd/mkosi/issues/1841.
+            "index=off",
+            "metacopy=off"
+        ]
 
         # userxattr is only supported on overlayfs since kernel 5.11
         if GenericVersion(platform.release()) >= GenericVersion("5.11"):