Omer Frenkel has submitted this change and it was merged.
Change subject: core: remove memory volumes without tasks on import vm failure
......................................................................
core: remove memory volumes without tasks on import vm failure
On failure to import vm with snapshots that contain memory state, we
remove the imported memory volumes.
This patch fix a bug which is relevant for the memory volumes removal in
this scenario, that was caused by not adding the tasks properly (not
override the getTaskType method in CommandBase). The solution is to
remove those imported memory volumes without tasks, in a similar way to
the solution done for I3830992f46b2c3fbea47e69f21a6711718408e95.
Change-Id: I596ea3164c70b6c2a4e2c161737421259457a179
Bug-Url:
https://bugzilla.redhat.com/1015080
Signed-off-by: Arik Hadas <ahadas(a)redhat.com>
---
M
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ImportVmCommand.java
1 file changed, 23 insertions(+), 6 deletions(-)
Approvals:
Omer Frenkel: Looks good to me, approved
Arik Hadas: Verified
--
To view, visit
http://gerrit.ovirt.org/19769
To unsubscribe, visit
http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I596ea3164c70b6c2a4e2c161737421259457a179
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Arik Hadas <ahadas(a)redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com>
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server