Change in ovirt-engine[master]: core: change the order of checks in canDoAction of migrate v...

ofrenkel at redhat.com ofrenkel at redhat.com
Thu Apr 25 13:14:01 UTC 2013


Omer Frenkel has submitted this change and it was merged.

Change subject: core: change the order of checks in canDoAction of migrate vm
......................................................................


core: change the order of checks in canDoAction of migrate vm

Change the order in which the checks whether a snapshot is being taken
for the VM or whether one of its disks is locked are carried on, so that
the snapshot check will be made before, as in that case the disks of the
VM are also locked but this check produces a more specific can-do-action
error for the user.

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

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


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I3e7cacc97fddd48e16564fda0911e7c9604fc731
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: Omer Frenkel <ofrenkel at redhat.com>



More information about the Engine-commits mailing list