Change in ovirt-engine[master]: core: remove memory volumes without tasks on import vm failure

ofrenkel at redhat.com ofrenkel at redhat.com
Thu Oct 3 13:38:03 UTC 2013


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 at 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 at redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas at redhat.com>
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek at redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofrenkel at redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzaslavs at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list