Change in ovirt-engine[master]: core: improve stop of suspended vm

[ahadas at redhat.com]

Arik Hadas has submitted this change and it was merged.

Change subject: core: improve stop of suspended vm
......................................................................


core: improve stop of suspended vm

Suspended VM is a VM that is not running but its memory from
previous run was saved so it would be restore next time the
VM will run.

Stop of a suspeneded VM basically means that all we need to
do is to change its status to Down and remove its memory
volumes. However, currently we do not change the status to
Down until the memory volumes are removed.

This patch improves this flow by changing the order of these
operations: first change the status of the VM to Down and
only then remove the memory volumes ('fire and forget').

Change-Id: I76445cf1bd5de5f8129d334d02ed15a415c06848
Signed-off-by: Arik Hadas <ahadas at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/StopVmCommandBase.java
1 file changed, 8 insertions(+), 80 deletions(-)

Approvals:
  Jenkins CI: Passed CI tests
  Omer Frenkel: Looks good to me, approved
  Arik Hadas: Verified



-- 
To view, visit https://gerrit.ovirt.org/37672
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I76445cf1bd5de5f8129d334d02ed15a415c06848
Gerrit-PatchSet: 6
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: Jenkins CI
Gerrit-Reviewer: Liron Aravot <laravot at redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofrenkel at redhat.com>
Gerrit-Reviewer: automation at ovirt.org