Dan Kenigsberg has submitted this change and it was merged. Change subject: ResourceManagerLock: Disable autoRelease ......................................................................
ResourceManagerLock: Disable autoRelease When acquiring ResourceManagerLocks we do not store a reference to the resource (prefering instead to use the ResourceManager instance methods). Unfortunately the underlying resource has autoRelease behavior on by default when the object is destroyed. This is causing the lock to be released immediately after it is acquired! Disable autoRelease to get the correct behavior. Change-Id: Ie70dca0079f773362bfebe6b9a08677c94314e68 Signed-off-by: Adam Litke <ali...@redhat.com> Reviewed-on: https://gerrit.ovirt.org/63967 Continuous-Integration: Jenkins CI Reviewed-by: Nir Soffer <nsof...@redhat.com> --- M vdsm/storage/resourceManager.py 1 file changed, 5 insertions(+), 1 deletion(-) Approvals: Nir Soffer: Looks good to me, approved Adam Litke: Verified Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/63967 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie70dca0079f773362bfebe6b9a08677c94314e68 Gerrit-PatchSet: 2 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Adam Litke <ali...@redhat.com> Gerrit-Reviewer: Adam Litke <ali...@redhat.com> Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Nir Soffer <nsof...@redhat.com> Gerrit-Reviewer: gerrit-hooks <automat...@ovirt.org> _______________________________________________ vdsm-patches mailing list -- vdsm-patches@lists.fedorahosted.org To unsubscribe send an email to vdsm-patches-le...@lists.fedorahosted.org
