Change in ovirt-engine[master]: core: refactor run vm as stateless

ahadas at redhat.com ahadas at redhat.com
Wed May 4 11:18:18 UTC 2016


Arik Hadas has submitted this change and it was merged.

Change subject: core: refactor run vm as stateless
......................................................................


core: refactor run vm as stateless

Minor refactoring in RunVmCommand related to run as stateless:
Instead of checking if the VM already has stateless snapshot inside
'statelessVmTreatment' method after the flow was set to
CREATE_STATELESS_IMAGES and in that case remove the snapshot, we are
now checking it in the getFlow method and if the VM already contains
stateless snapshot then the flow is set to REMOVE_STATELESS_IMAGES.

Change-Id: I350e263bb326c8d3c6be78b2d5b2bd1a3bd0535f
Signed-off-by: Arik Hadas <ahadas at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RunVmCommand.java
M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/RunVmCommandTest.java
2 files changed, 48 insertions(+), 29 deletions(-)

Approvals:
  Shahar Havivi: Looks good to me, approved
  Jenkins CI: Passed CI tests
  Arik Hadas: Verified



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I350e263bb326c8d3c6be78b2d5b2bd1a3bd0535f
Gerrit-PatchSet: 34
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: Jenkins CI RO
Gerrit-Reviewer: Michal Skrivanek <mskrivan at redhat.com>
Gerrit-Reviewer: Shahar Havivi <shavivi at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>



More information about the Engine-commits mailing list