[Engine-patches] Change in ovirt-engine[master]: core: VM status is updated although the last operation on it...

emesika at redhat.com emesika at redhat.com
Wed May 2 17:43:50 EDT 2012


Eli Mesika has submitted this change and it was merged.

Change subject: core: VM status is updated although the last operation on it has not finished yet
......................................................................


core: VM status is updated although the last operation on it has not finished yet

Problem was that Resume command adds the vm to the _asyncRunningVms collection in ResourceMnager.
Then in the canExecute method of CreateVdsCommand the vm is checked for being in that collection and if it is
found , than the status of the VM is set to UP before the command even starts to execute.

The solution is to take and return the curent status as reflected in the vm_dynamic table.

Change-Id: I1e97372d23dd4edcba5a0e9147beabef81bc70e2
Signed-off-by: Eli Mesika <emesika at redhat.com>
---
M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/CreateVmVDSCommand.java
1 file changed, 5 insertions(+), 6 deletions(-)

Approvals:
  Eli Mesika: Verified; Looks good to me, approved


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I1e97372d23dd4edcba5a0e9147beabef81bc70e2
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Eli Mesika <emesika at redhat.com>
Gerrit-Reviewer: Eli Mesika <emesika at redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofrenkel at redhat.com>


More information about the Engine-patches mailing list