From f65c4edf8372da33f5df7fe3e8098dcc5a3a6cc6 Mon Sep 17 00:00:00 2001
From: "Simon L." <szaimen@e.mail.de>
Date: Sun, 19 Jan 2025 09:54:17 +0100
Subject: [PATCH] rsyncbackup: do not unmount any drives if backup failed
 because pending snashot exists

Signed-off-by: Simon L. <szaimen@e.mail.de>
---
 not-supported/rsyncbackup.sh | 16 +++++++++-------
 1 file changed, 9 insertions(+), 7 deletions(-)

diff --git a/not-supported/rsyncbackup.sh b/not-supported/rsyncbackup.sh
index 1f68e00b03..ab21a7a088 100644
--- a/not-supported/rsyncbackup.sh
+++ b/not-supported/rsyncbackup.sh
@@ -19,7 +19,6 @@ debug_mode
 root_check
 
 # Variables
-LVM_MOUNT="/system"
 START_TIME=$(date +%s)
 CURRENT_DATE=$(date --date @"$START_TIME" +"%Y%m%d_%H%M%S")
 CURRENT_DATE_READABLE=$(date --date @"$START_TIME" +"%d.%m.%Y - %H:%M:%S")
@@ -48,14 +47,17 @@ show_drive_usage() {
 send_error_mail() {
     if [ -d "$BACKUP_TARGET_DIRECTORY" ]
     then
-        inform_user "$ICyan" "Unmounting the off-shore backup drive..."
-        umount "$BACKUP_MOUNTPOINT"
+        if [ -z "$DO_NOT_UMOUNT_BACKUP_DRIVES" ]
+        then
+            inform_user "$ICyan" "Unmounting the offshore backup drive..."
+            umount "$BACKUP_MOUNTPOINT"
+        fi
     fi
     if [ -d "$BACKUP_SOURCE_DIRECTORY" ]
     then
-        if [ -z "$DO_NOT_UMOUNT_DAILY_BACKUP_DRIVE" ]
+        if [ -z "$DO_NOT_UMOUNT_BACKUP_DRIVES" ]
         then
-            inform_user "$ICyan" "Unmounting the backup drive..."
+            inform_user "$ICyan" "Unmounting the daily backup drive..."
             umount "$BACKUP_SOURCE_MOUNTPOINT"
         fi
     fi
@@ -124,7 +126,7 @@ fi
 # Check if pending snapshot is existing and cancel the backup in this case.
 if does_snapshot_exist "NcVM-snapshot-pending"
 then
-    DO_NOT_UMOUNT_DAILY_BACKUP_DRIVE=1
+    DO_NOT_UMOUNT_BACKUP_DRIVES=1
     msg_box "The snapshot pending does exist. Can currently not proceed.
 Please try again later.\n
 If you are sure that no update or backup is currently running, you can fix this by rebooting your server."
@@ -193,7 +195,7 @@ fi
 # Check if pending snapshot is existing and cancel the backup in this case.
 if does_snapshot_exist "NcVM-snapshot-pending"
 then
-    DO_NOT_UMOUNT_DAILY_BACKUP_DRIVE=1
+    DO_NOT_UMOUNT_BACKUP_DRIVES=1
     msg_box "The snapshot pending does exist. Can currently not proceed.
 Please try again later.\n
 If you are sure that no update or backup is currently running, you can fix this by rebooting your server."