Arik Hadas has submitted this change and it was merged. Change subject: core: Add VM to touchedVms only once ...................................................................... core: Add VM to touchedVms only once If lockTouchedVm() is called several times for the same VM in the same thread, the VM ID must be added to touchedVms only once, when the lock is actually taken for the first time. Change-Id: I1e6119ae81d84250083ec390d757ff00baa439d7 Signed-off-by: Shmuel Melamud <smelamud@redhat.com> --- M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VmDevicesMonitoring.java 1 file changed, 11 insertions(+), 5 deletions(-) Approvals: Shmuel Leib Melamud: Verified Arik Hadas: Looks good to me, approved; Passed CI tests Objections: Jenkins CI: Failed CI tests -- To view, visit https://gerrit.ovirt.org/53161 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1e6119ae81d84250083ec390d757ff00baa439d7 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Shmuel Leib Melamud <smelamud@redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas@redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Liron Aravot <laravot@redhat.com> Gerrit-Reviewer: Shmuel Leib Melamud <smelamud@redhat.com> Gerrit-Reviewer: gerrit-hooks <automation@ovirt.org>