From: Alan J. Pippin <alan@pippins.net>
Date: Mon, 3 Feb 2020 21:01:29 +0000 (-0700)
Subject: Fixed bug that caused empty error messages to be emailed back
X-Git-Url: http://git.pippins.net/%27%20.%20%24this-%3Ephpgw_js_url%20.%20%27/jscalendar/calendar-win2k-cold-1.css?a=commitdiff_plain;h=3cae66d2b4a3eba94cb0765db85ff5cdc0ac194a;p=zfs-ubuntu%2F.git

Fixed bug that caused empty error messages to be emailed back

Change-Id: I28685c64435343bc48c938cb422ad06373de01f6
---

diff --git a/zfs-replicate-all b/zfs-replicate-all
index 66949df..f5e23e7 100755
--- a/zfs-replicate-all
+++ b/zfs-replicate-all
@@ -50,16 +50,15 @@ cleanup_and_exit() {
 }
 trap cleanup_and_exit INT
 
-fatal() {
+fatal_and_exit() {
   # echo message to terminal
   echo -e 2>&1 "$1"
   # send email notification
-  echo -e "$1" | $mailx -s "zfs-replicate-all on $hostname failed" "$mailto"
-}
-
-fatal_and_exit() {
-  fatal $*
-  exit 1
+  echo -e "$1" | $mailx -s "zfs-replicate-all on $hostname failed" "$2"
+  # exit with bad return code unless 3rd argument was defined that says not to
+  if [[ -z "$3" ]]; then
+    exit 1
+  fi
 }
 
 # This function executes the replicate command and checks the stoptime
@@ -153,7 +152,7 @@ if [[ $import_export_backup_pool == 1 ]]; then
     zpool export $backup_pool
     if [ $? != 0 ]; then
       lsof /$backup_pool/*
-      fatal "-E- unable to export the local backup pool $backup_pool on $hostname" "$mailto"
+      fatal_and_exit "-E- unable to export the local backup pool $backup_pool on $hostname" "$mailto" 0
     fi
   fi
 fi