Change in ovirt-engine[master]: core: correct lock in UpdateVmDynamicData

ahadas at redhat.com ahadas at redhat.com
Wed Jul 20 07:37:49 UTC 2016


Arik Hadas has submitted this change and it was merged.

Change subject: core: correct lock in UpdateVmDynamicData
......................................................................


core: correct lock in UpdateVmDynamicData

This patch fixes the following bug: UpdateVmDynamicData is supposed to take VM
lock so the monitoring won't override the changes, but instead of locking the
VM to be updated, it always locked the blank template (empty guid).

Change-Id: I94638190a94082f7dd42019ceabc6ed1ccb53cad
Signed-off-by: Arik Hadas <ahadas at redhat.com>
---
M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/vdscommands/UpdateVmDynamicDataVDSCommandParameters.java
1 file changed, 1 insertion(+), 0 deletions(-)

Approvals:
  Shahar Havivi: Looks good to me, approved
  Arik Hadas: Verified; Passed CI tests



-- 
To view, visit https://gerrit.ovirt.org/61061
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I94638190a94082f7dd42019ceabc6ed1ccb53cad
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: Jenkins CI
Gerrit-Reviewer: Shahar Havivi <shavivi at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>



More information about the Engine-commits mailing list