B #7400: Fix Thin Pool extend during snap operations (#3966)

Signed-off-by: Guillermo Ramos <gramos@opennebula.io>

Author: 1gramos

src/tm_mad/fs_lvm_ssh/snap_create

@@ -88,8 +88,20 @@ SNAP_CREATE_CMD=$(cat <<EOF
 
     ${SYNC}
     ${SUDO} ${LVSCAN}
-    $SUDO $LVEXTEND -L +${SIZE}M $VG_NAME/$POOL_NAME
+
     $SUDO $LVCREATE -s -n $SNAP_NAME $VG_NAME/$LV_NAME
+
+    # Extend the pool as needed
+    pool_size="\$($SUDO $LVS --nosuffix --noheadings --units B -o lv_size $VG_NAME/$POOL_NAME)"
+    lvs_size=0
+    for lv in \$($SUDO $LVS --noheading -o lv_name -S 'vg_name = $VG_NAME && pool_lv = $POOL_NAME'); do
+        lv_size="\$($SUDO $LVS --nosuffix --noheadings --units B -o lv_size $VG_NAME/\$lv)"
+        lvs_size=\$((lvs_size + lv_size))
+    done
+    if [ "\$lvs_size" -gt "\$pool_size" ]; then
+        $SUDO $LVEXTEND -L \${lvs_size}B $VG_NAME/$POOL_NAME
+    fi
+
     ${SYNC}
 EOF
 )

src/tm_mad/fs_lvm_ssh/snap_create_live

@@ -84,8 +84,20 @@ SNAP_CREATE_CMD=$(cat <<EOF
         trap "virsh -c $LIBVIRT_URI domfsthaw $DEPLOY_ID || true" EXIT
         ${SYNC}
         ${SUDO} ${LVSCAN}
-        $SUDO $LVEXTEND -L +${SIZE}M $VG_NAME/$POOL_NAME
+
         $SUDO $LVCREATE -s -n $SNAP_NAME $VG_NAME/$LV_NAME
+
+        # Extend the pool as needed
+        pool_size="\$($SUDO $LVS --nosuffix --noheadings --units B -o lv_size $VG_NAME/$POOL_NAME)"
+        lvs_size=0
+        for lv in \$($SUDO $LVS --noheading -o lv_name -S 'vg_name = $VG_NAME && pool_lv = $POOL_NAME'); do
+            lv_size="\$($SUDO $LVS --nosuffix --noheadings --units B -o lv_size $VG_NAME/\$lv)"
+            lvs_size=\$((lvs_size + lv_size))
+        done
+        if [ "\$lvs_size" -gt "\$pool_size" ]; then
+            $SUDO $LVEXTEND -L \${lvs_size}B $VG_NAME/$POOL_NAME
+        fi
+
         ${SYNC}
     else
         echo "Unable to freeze VM; aborting"

src/tm_mad/fs_lvm_ssh/snap_revert

@@ -89,6 +89,18 @@ SNAP_REVERT_CMD=$(cat <<EOF
         ${SUDO} ${LVSCAN}
         $SUDO $LVREMOVE -y $VG_NAME/$LV_NAME
         $SUDO $LVCREATE -s -n $LV_NAME $VG_NAME/$SNAP_NAME
+
+        # Extend the pool as needed
+        pool_size="\$($SUDO $LVS --nosuffix --noheadings --units B -o lv_size $VG_NAME/$POOL_NAME)"
+        lvs_size=0
+        for lv in \$($SUDO $LVS --noheading -o lv_name -S 'vg_name = $VG_NAME && pool_lv = $POOL_NAME'); do
+            lv_size="\$($SUDO $LVS --nosuffix --noheadings --units B -o lv_size $VG_NAME/\$lv)"
+            lvs_size=\$((lvs_size + lv_size))
+        done
+        if [ "\$lvs_size" -gt "\$pool_size" ]; then
+            $SUDO $LVEXTEND -L \${lvs_size}B $VG_NAME/$POOL_NAME
+        fi
+
         # Next two commands could be merged, but is advised against by LVM
         $SUDO $LVCHANGE -kn $VG_NAME/$LV_NAME
         $SUDO $LVCHANGE -ay $VG_NAME/$LV_NAME