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(a)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(a)redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Jenkins CI RO
Gerrit-Reviewer: Michal Skrivanek <mskrivan(a)redhat.com>
Gerrit-Reviewer: Shahar Havivi <shavivi(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>