return success if snapshots not sent because health check fails.

bin/zsnapper

@@ -369,14 +369,12 @@ def main():
         try:
             if remote in remote_snapshots:
                 remote_snapshots[remote] = zsnaplib.get_snapshots(zfs_cmd)
-        except zsnaplib.ZFSSnapshotError:
+        except zsnaplib.ZFSSnapshotError as e:
             del remote_snapshots[remote]
-            log.warning("Could not refresh snapshots on {}".format(remote))
+            log.warning("Could not refresh snapshots on {}: {}".format(remote, e))
     snapshots = zsnaplib.get_snapshots(local_zfs_cmd)
 
     failed_send = send_snapshots(fslist, snapshots, config)
-    if failed_send:
-        ret = RET_CODES['ERROR']
     failed_snapshots.update(failed_send)
     for remote in remotes.keys():
         failed_send = send_snapshots(
@@ -397,6 +395,7 @@ def main():
                 failed_snapshots)
 
     os.remove(lockfile)
+    return ret
 
 if __name__ == '__main__':
     log = logging.getLogger(LOGGER)

zsnaplib/__init__.py

@@ -83,7 +83,7 @@ def get_filesystems(zfs_cmd):
 
     for row in out.splitlines():
         row = row.decode('UTF-8')
-        ret.add(row.split()[0])
+        ret.add(row.split('\t')[0])
     return ret
 
 
@@ -93,7 +93,7 @@ def get_snapshots(zfs_cmd):
     snapshots = {}
 
     for row in out.splitlines():
-        row = row.decode('UTF-8').split()[0]
+        row = row.decode('UTF-8').split('\t')[0]
         res = re_snapshot.match(row)
         if res:
             d = datetime.datetime.strptime(res.group(2), time_format)