fix(backupVm): remove XVA on failure (#626)

Fixes vatesfr/xo-web#2524
2017-12-11 14:40:52 +01:00
parent 043fc5cdfc
commit d9ad53d954
1 changed files with 5 additions and 1 deletions
--- a/src/xo-mixins/backups.js
+++ b/src/xo-mixins/backups.js
@@ -972,8 +972,12 @@ export default class {
    return this._backupVm(vm, handler, file, {compress, onlyMetadata})
  }

-  async _backupVm (vm, handler, file, {compress, onlyMetadata}) {
+  @deferrable.onFailure
+  async _backupVm ($onFailure, vm, handler, file, {compress, onlyMetadata}) {
    const targetStream = await handler.createOutputStream(file)
+    $onFailure.call(handler, 'unlink', file)
+    $onFailure.call(targetStream, 'close')
+
    const promise = eventToPromise(targetStream, 'finish')

    const sourceStream = await this._xo.getXapi(vm).exportVm(vm._xapiId, {