Finalize local export only if successful
authorHrvoje Ribicic <riba@google.com>
Thu, 12 Feb 2015 17:34:47 +0000 (17:34 +0000)
committerHrvoje Ribicic <riba@google.com>
Wed, 18 Feb 2015 18:20:26 +0000 (19:20 +0100)
commit1ef87af36b267c71115ebb52b58e7590278e92a9
tree49a0e345868530197caf8ea36dd6c828cfb9d320
parent1ab5e9f5e05dcd6f63d46779df0023fdebcf9b01
Finalize local export only if successful

Before commit 44247302, a failure in the transfer of a single disk
during an export was changed to make the entire transfer unsuccessful.
What did not change was an invocation of export finalization, which
removes the old backup if present and replaces it with whatever the
result of the transfer was. As a result, a failed backup of an instance
could destroy a previous backup.

This patch prevents the finalization from taking place - this leaves
some data lying around in a ".new" directory, but at least the old
backup is not deleted.

Signed-off-by: Hrvoje Ribicic <riba@google.com>
Reviewed-by: Helga Velroyen <helgav@google.com>
lib/masterd/instance.py