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 <[email protected]>
---
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 <[email protected]>
Gerrit-Reviewer: Eli Mesika <[email protected]>
Gerrit-Reviewer: Omer Frenkel <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to