storage: when changing format of a new partition, don't require wipe

When dealing with a new partition, changing the format from e.g. ext4 to unformatted should not require a wipe value. There is nothing yet to wipe. OTOH it is still necessary for existing partitions. Signed-off-by: Olivier Gayot <[email protected]>
canonical · Jan 27, 2025 · 60693ec · 60693ec
1 parent 78b2719
commit 60693ec
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/subiquity/server/controllers/filesystem.py b/subiquity/server/controllers/filesystem.py
@@ -1404,8 +1404,10 @@ async def v2_edit_partition_POST(
         # format="".
         if data.partition.format is not None:
             if data.partition.format != partition.original_fstype():
-                if data.partition.wipe is None:
-                    raise ValueError("changing partition format requires a wipe value")
+                if data.partition.wipe is None and partition.preserve:
+                    raise ValueError(
+                        "changing format of existing partition requires a wipe value"
+                    )
             spec["fstype"] = data.partition.format
         if data.partition.size is not None:
             spec["size"] = data.partition.size