PR#103: Fix resizing underlying filesystem together with the LV - arm-image-installer

arm-image-installer

#103 Fix resizing underlying filesystem together with the LV

Closed 2 months ago by pwhalen. Opened 5 months ago by hricky.

hricky/arm-image-installer fix-resizefs into main

Clean up redundant XFS filesystem resizing

Hristo Marinov • 5 months ago

eb43a3b

Clean up XFS file system resizing conditional statement

Hristo Marinov • 5 months ago

454cd57

Fix resizing underlying filesystem together with the LV

Hristo Marinov • 5 months ago

8d3df35

arm-image-installer

file modified

+2 -6

		`@@ -467,16 +467,12 @@`
		`if [ "$LVM_NAME" != "" ]; then`
		`vgchange -a y $LVM_NAME > /dev/null 2>&1`
		`pvresize "$ROOTPART"`
		`- lvextend -l +100%FREE /dev/$LVM_NAME/root`
		`+ lvextend -l +100%FREE -ry /dev/$LVM_NAME/root`
		`ROOTPART="/dev/$LVM_NAME/root"`
		`fi`
		`fi`

		`- if [ "$FS_TYPE" = "xfs" ] && [ "$LVM_NAME" != "" ]; then`
		`- mkdir /tmp/root > /dev/null 2>&1`
		`- mount "$ROOTPART" /tmp/root > /dev/null 2>&1`
		`- xfs_growfs /tmp/root`
		`- elif [ "$FS_TYPE" = "btrfs" ]; then`
		`+ if [ "$FS_TYPE" = "btrfs" ]; then`
		`mkdir /tmp/root > /dev/null 2>&1`
		`mount "$ROOTPART" /tmp/root > /dev/null 2>&1`
		`btrfs filesystem resize max /tmp/root`

hricky commented 5 months ago

When arm-image-installer is executed with the --resizefs option, in the case of an image that uses LVM (like Fedora Server), FS_TYPE gets LVM2_member as the value (Line 407). This way Line 475 is not triggered and the file system is not resized.

By adding -ry to the lvextend command (Line 470), it automatically resizes the file system along with the LV.