Federico Simoncelli has posted comments on this change.
Change subject: Reuse the stored pool host id on reconstructMaster
......................................................................
Patch Set 1: (2 inline comments)
....................................................
File vdsm/storage/sp.py
Line 203: def _acquireHostId(self, sdUUID, isValid):
It's quite a while that I try to remove it. Please refer to:
http://gerrit.ovirt.org/#/c/3929/
Line 496: def _acquireTemporaryClusterLock(self, msdUUID, safeLease, hostId=None):
We might want to leave this alone (using it only in create for now and eventually get rid
of it later) and then require the proper hostId in reconstructMaster..
--
To view, visit
http://gerrit.ovirt.org/4965
To unsubscribe, visit
http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I44ff6d04220032ac1f6bd0a7819132103050b516
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Federico Simoncelli <fsimonce(a)redhat.com>
Gerrit-Reviewer: Ayal Baron <abaron(a)redhat.com>
Gerrit-Reviewer: Federico Simoncelli <fsimonce(a)redhat.com>