Fixed error checking for sqldump and redis dump. Used -T param in exec

root/scripts/backup.sh

@@ -197,7 +197,7 @@ function cleanup {
     ## restart mailflow docker containers
     # start and verify postfix
     operateDocker start postfix
-    if [ "$dockerResult" = "true" ]; then
+    if [ "$dockerResultState" = "true" ]; then
         echo -e "${info}${stamp} -- [INFO] Postfix container is running --" \
             "${normal}" >> "$logFile"
     else
@@ -205,7 +205,7 @@ function cleanup {
     fi
     # start and verify dovecot
     operateDocker start dovecot
-    if [ "$dockerResult" = "true" ]; then
+    if [ "$dockerResultState" = "true" ]; then
         echo -e "${info}${stamp} -- [INFO] Dovecot container is running --" \
             "${normal}" >> "$logFile"
     else
@@ -221,14 +221,19 @@ if [ "$1" = "stop" ]; then
         >> "$logFile"
     docker-compose stop --timeout ${dockerStopTimeout} ${2}-mailcow \
         2>> "$logFile"
-    # verify
+    # verify container stopped (should return true)
-    dockerResult=$(docker inspect -f '{{ .State.ExitCode }}' ${COMPOSE_PROJECT_NAME}_${2}-mailcow_1)
+    dockerResultState=$(docker inspect -f '{{ .State.Running }}' \
+        ${COMPOSE_PROJECT_NAME}_${2}-mailcow_1)
+    # verify clean stop (exit code 0)
+    dockerResultExit=$(docker inspect -f '{{ .State.ExitCode }}' \
+        ${COMPOSE_PROJECT_NAME}_${2}-mailcow_1)
 elif [ "$1" = "start" ]; then
     echo -e "${op}${stamp} Starting ${2}-mailcow container...${normal}" \
         >> "$logFile"
     docker-compose start ${2}-mailcow 2>> "$logFile"
     # verify
-    dockerResult=$(docker inspect -f '{{ .State.Running }}' ${COMPOSE_PROJECT_NAME}_${2}-mailcow_1)
+    dockerResultState=$(docker inspect -f '{{ .State.Running }}' \
+        ${COMPOSE_PROJECT_NAME}_${2}-mailcow_1)
 fi
 }
 
@@ -438,6 +443,9 @@ echo -e "${info}${stamp} -- [INFO] using ${lit}${dockerComposeFilePath}" \
 ### Import additional variables from mailcow configuration file
 source "${mailcowConfigFilePath}"
 
+### Export PATH so this script can access all docker and docker-compose commands
+export PATH=/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin
+
 ### Export logFile variable for use by Borg
 export logFile="$logFile"
 
@@ -524,7 +532,7 @@ cd "$mailcowPath"
 ## Stop postfix-mailcow container
 operateDocker stop postfix
 # process result
-if [ "$dockerResult" -eq 0 ]; then
+if [ "$dockerResultState" = "false" ] && [ "$dockerResultExit" -eq 0 ]; then
     echo -e "${info}${stamp} -- [INFO] Postfix container stopped --${normal}" \
         >> "$logFile"
 else
@@ -535,7 +543,7 @@ fi
 ## Stop dovecot-mailcow container
 operateDocker stop dovecot
 # process result
-if [ "$dockerResult" -eq 0 ]; then
+if [ "$dockerResultState" = "false" ] && [ "$dockerResultExit" -eq 0 ]; then
     echo -e "${info}${stamp} -- [INFO] Dovecot container stopped --${normal}" \
         >> "$logFile"
 else
@@ -548,9 +556,9 @@ fi
 ### Dump SQL
 echo -e "${op}${stamp} Dumping mailcow SQL database...${normal}" >> "$logFile"
 docker-compose exec mysql-mailcow mysqldump --default-character-set=utf8mb4 -u${DBUSER} -p${DBPASS} ${DBNAME} > "$sqlDumpDir/$sqlDumpFile" 2>> "$logFile"
-checkResult=$(docker-compose exec mysql-mailcow echo "$?")
+checkResult=$(docker-compose exec -T mysql-mailcow echo $?)
 # verify sql dump was successful
-if [ "$checkResult"=0 ]; then
+if [ "$checkResult" = 0 ]; then
     echo -e "${ok}${stamp} -- [SUCCESS] SQL successfully dumped --${normal}" \
         >> "$logFile"
 else
@@ -561,9 +569,9 @@ fi
 ### Save redis state
 echo -e "${op}${stamp} Saving redis state information...${normal}" >> "$logFile"
 docker-compose exec redis-mailcow redis-cli save >> "$logFile" 2>&1
-checkResult=$(docker-compose exec redis-mailcow echo "$?")
+checkResult=$(docker-compose exec -T redis-mailcow echo "$?")
 # Verify save was successful
-if [ "$checkResult"=0 ]; then
+if [ "$checkResult" = 0 ]; then
     echo -e "${ok}${stamp} -- [SUCCESS] redis state saved --${normal}" \
         >> "$logFile"
 else