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(a)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(a)redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Shahar Havivi <shavivi(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>