fix(xo-server/xapi#exportVm): destroy snapshot on getResource failure

2018-10-04 17:57:15 +02:00
parent 90de47d708
commit 65307e5bc7
1 changed files with 2 additions and 3 deletions
--- a/packages/xo-server/src/xapi/index.js
+++ b/packages/xo-server/src/xapi/index.js
@@ -763,9 +763,8 @@ export default class Xapi extends XapiBase {
    })

    if (useSnapshot) {
-      promise.then(_ =>
-        _.task::pFinally(() => this.deleteVm(exportedVm)::ignoreErrors())
-      )
+      const destroySnapshot = () => this.deleteVm(exportedVm)::ignoreErrors()
+      promise.then(_ => _.task::pFinally(destroySnapshot), destroySnapshot)
    }

    return promise