Dan Kenigsberg has submitted this change and it was merged.
Change subject: Group the cluster lock code into a new SP method ......................................................................
Group the cluster lock code into a new SP method
Both the create and reconstructMaster methods were sharing a very similar code for temporarly acquiring the cluster lock.
Change-Id: I981d84af7816ccd2db64eb5beb440e9f7637f734 --- M vdsm/storage/safelease.py M vdsm/storage/sd.py M vdsm/storage/sp.py 3 files changed, 36 insertions(+), 25 deletions(-)
Approvals: Federico Simoncelli: Verified Dan Kenigsberg: Looks good to me, approved Igor Lvovsky: Looks good to me, but someone else must approve
-- To view, visit http://gerrit.ovirt.org/603 To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged Gerrit-Change-Id: I981d84af7816ccd2db64eb5beb440e9f7637f734 Gerrit-PatchSet: 6 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Federico Simoncelli fsimonce@redhat.com Gerrit-Reviewer: Dan Kenigsberg danken@redhat.com Gerrit-Reviewer: Eduardo ewarszaw@redhat.com Gerrit-Reviewer: Federico Simoncelli fsimonce@redhat.com Gerrit-Reviewer: Igor Lvovsky ilvovsky@redhat.com Gerrit-Reviewer: Saggi Mizrahi smizrahi@redhat.com