Change in vdsm[master]: clusterLock: fix returned retval

danken at redhat.com danken at redhat.com
Thu May 7 15:08:13 UTC 2015


Dan Kenigsberg has submitted this change and it was merged.

Change subject: clusterLock: fix returned retval
......................................................................


clusterLock: fix returned retval

Change Id4b9227773e44d48b21aa4eaba7c7fe1ab599931 added accidently
a third element to the returned tuple which breaks the expected
behavior.

This patch restores the previous returned value and adds parantheses
to clarify what is returned.

Change-Id: I21611beaeb430c04ff5678189733d54cb1df6a84
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1218325
Signed-off-by: Liron Aravot <laravot at redhat.com>
Reviewed-on: https://gerrit.ovirt.org/40652
Reviewed-by: Federico Simoncelli <fsimonce at redhat.com>
---
M vdsm/storage/clusterlock.py
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  Federico Simoncelli: Looks good to me, approved
  Liron Aravot: Verified



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I21611beaeb430c04ff5678189733d54cb1df6a84
Gerrit-PatchSet: 5
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Liron Aravot <laravot at redhat.com>
Gerrit-Reviewer: Adam Litke <alitke at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Federico Simoncelli <fsimonce at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Liron Aravot <laravot at redhat.com>
Gerrit-Reviewer: Nir Soffer <nsoffer at redhat.com>
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo at redhat.com>
Gerrit-Reviewer: automation at ovirt.org


More information about the vdsm-patches mailing list