Added check to not export pools if zfs-replicate or zfs-scrub process is running
authorAlan J. Pippin <ajp@pippins.net>
Thu, 13 Mar 2014 13:48:58 +0000 (07:48 -0600)
committerAlan J. Pippin <ajp@pippins.net>
Thu, 13 Mar 2014 13:52:33 +0000 (07:52 -0600)
zfs-replicate-all
zfs-scrub

index afd30b89c2f280ddcb6ec796d9f6dda6ca097219..62a0ce4d10f60a78cde674427c974341596c413a 100755 (executable)
@@ -130,9 +130,13 @@ done
 
 # Export the local pool if told to do so
 if [[ $import_export_backup_pool == 1 ]]; then
-  zpool export $backup_pool
+  # Don't export the pool if there is a currently running zfs-scrub operation
+  ps -ef | grep -q "zfs-scrub ${backup_pool}" | grep -v grep
   if [ $? != 0 ]; then
-    fatal_and_exit "-E- unable to export the local backup pool $backup_pool on $hostname" "$mailto"
+    zpool export $backup_pool
+    if [ $? != 0 ]; then
+      fatal_and_exit "-E- unable to export the local backup pool $backup_pool on $hostname" "$mailto"
+    fi
   fi
 fi
 
index 495adf4f9b7ab4c795727c7e3e810dcbe092e4be..840987693424e931cdb3472046d57ac189e2cca5 100755 (executable)
--- a/zfs-scrub
+++ b/zfs-scrub
@@ -90,10 +90,14 @@ do
 
   # Export the local pool if told to do so
   if [[ $import_export_backup_pool == 1 ]] && [[ "$i" =~ "$backup_pool" ]]; then
-    zpool export $i
+    # Don't export if there is a currently running zfs-replicate operation
+    ps -ef | grep -q "zfs-replicate" | grep -v grep
     if [ $? != 0 ]; then
-      echo "-E- unable to export the local pool $i"
-      zpool status $i | $mailx -s "zpool scrub on $hostname unable to export the local pool $i" "$mailto"
+      zpool export $i
+      if [ $? != 0 ]; then
+        echo "-E- unable to export the local pool $i"
+        zpool status $i | $mailx -s "zpool scrub on $hostname unable to export the local pool $i" "$mailto"
+      fi
     fi
   fi