oetiker / znapzend

zfs backup with remote capabilities and mbuffer integration.
www.znapzend.org
GNU General Public License v3.0
608 stars 137 forks source link

Allow skipping of snap create/send on pre-command failure #316

Closed atj closed 6 years ago

atj commented 6 years ago

This commit implements two new boolean command line options:

These options cause znapzend to skip snapshot creation or replication if the respective pre-command returns a non-zero exit code or is killed by a signal.

atj commented 6 years ago

I'm receptive to comments and/or suggestions on this change. Particularly if someone can suggest some better option names!

coveralls commented 6 years ago

Coverage Status

Coverage decreased (-0.6%) to 87.322% when pulling 65c1ae30e8f83b9bd8f4551bcf82c41f1518dd69 on atj:feature/skip_on_pre_cmd_failure into 3c02d29578e94a6a5c19cd04fa3a82a61176035b on oetiker:master.

coveralls commented 6 years ago

Coverage Status

Coverage decreased (-0.6%) to 87.322% when pulling 65c1ae30e8f83b9bd8f4551bcf82c41f1518dd69 on atj:feature/skip_on_pre_cmd_failure into 3c02d29578e94a6a5c19cd04fa3a82a61176035b on oetiker:master.

coveralls commented 6 years ago

Coverage Status

Coverage decreased (-0.6%) to 87.322% when pulling 65c1ae30e8f83b9bd8f4551bcf82c41f1518dd69 on atj:feature/skip_on_pre_cmd_failure into 3c02d29578e94a6a5c19cd04fa3a82a61176035b on oetiker:master.

oetiker commented 6 years ago

I like it, @hadfl what do you think ?

hadfl commented 6 years ago

lgtm

gabviv73 commented 6 years ago

For me works correctly.