Change in ovirt-engine[master]: core: change can-do checks for resume action

ofrenkel at redhat.com ofrenkel at redhat.com
Wed Mar 6 16:11:33 UTC 2013


Omer Frenkel has submitted this change and it was merged.

Change subject: core: change can-do checks for resume action
......................................................................


core: change can-do checks for resume action

Currently resume is done as a special flow of run-vm command when the
previous state of the VM is paused. the can-do-action checks in that
case were the same as those that are made for regular run-vm action (for
VM which are down).

This patch changes the can-do-action checks in case of resume action
such that they will only check that the VM exists. there is no need to
do further checks as the VM was already created (thus was found to be
valid before) if its state is paused.

Change-Id: I772377045d5c6d694e00006c9b12e210c52ab353
Bug-Url: https://bugzilla.redhat.com/885994
Signed-off-by: Arik Hadas <ahadas at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RunVmCommand.java
1 file changed, 26 insertions(+), 13 deletions(-)

Approvals:
  Omer Frenkel: Verified; Looks good to me, approved


--
To view, visit http://gerrit.ovirt.org/12769
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I772377045d5c6d694e00006c9b12e210c52ab353
Gerrit-PatchSet: 3
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: Omer Frenkel <ofrenkel at redhat.com>
Gerrit-Reviewer: Roy Golan <rgolan at redhat.com>



More information about the Engine-commits mailing list