feffe/zsnapper
1
0
Fork 0
Code Issues Pull Requests Projects Releases 7 Wiki Activity

Fixed a bug that removed snapshots even if zfs send failed. Removed

Browse Source 
debug output
This commit is contained in:
Fredrik Eriksson 2017-06-07 19:02:37 +02:00
parent 8adf8f14f8
commit b40b098024
2 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
bin/zsnapper
View File

@ -210,6 +210,7 @@ def send_snapshots(fslist, snapshots, config, sudo):
log.info('{} successfully sent to remote'.format(fs)) log.info('{} successfully sent to remote'.format(fs))
except zsnaplib.ZFSSnapshotError as e: except zsnaplib.ZFSSnapshotError as e:
log.warning(e) log.warning(e)
failed_snapshots.add(fs)
return failed_snapshots return failed_snapshots
def weed_snapshots(fslist, snapshots, config, sudo, failed_snapshots): def weed_snapshots(fslist, snapshots, config, sudo, failed_snapshots):

1
zsnaplib/__init__.py
View File

@ -46,7 +46,6 @@ def do_zfs_command(args, sudo, pipecmd=None, zfs_cmd=[zfs_bin]):
(out, err) = ctrl_proc.communicate() (out, err) = ctrl_proc.communicate()
if ctrl_proc.returncode != 0: if ctrl_proc.returncode != 0:
print(proc.returncode)
raise ZFSSnapshotError('Failed to execute {}: {}'.format(cmd, err)) raise ZFSSnapshotError('Failed to execute {}: {}'.format(cmd, err))
return out return out