Ayal Baron has posted comments on this change.
Change subject: sp: split metadata transaction in createMaster
......................................................................
Patch Set 4:
(1 comment)
See patch set 4
....................................................
File vdsm/storage/sp.py
Line 695: self.stopMonitoringDomains()
Line 696: return True
Line 697:
Line 698: @unsecured
Line 699: def initMasterDomain(self, poolName, domain, masterVersion):
but it's not the master metadata, it's the pool metadata.
that is the differentiation between initMaster and this.
Since it is in the context of the pool then it could be just initMetadata
Line 700: self._getPoolMD(domain).update({
Line 701: PMDK_SPM_ID: SPM_ID_FREE,
Line 702: PMDK_LVER: LVER_INVALID,
Line 703: PMDK_MASTER_VER: masterVersion,
--
To view, visit http://gerrit.ovirt.org/22418
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I3b22af92b1f9a481be9af844b1acc47ae513d078
Gerrit-PatchSet: 4
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Federico Simoncelli <[email protected]>
Gerrit-Reviewer: Adam Litke <[email protected]>
Gerrit-Reviewer: Allon Mureinik <[email protected]>
Gerrit-Reviewer: Antoni Segura Puimedon <[email protected]>
Gerrit-Reviewer: Ayal Baron <[email protected]>
Gerrit-Reviewer: Federico Simoncelli <[email protected]>
Gerrit-Reviewer: Nir Soffer <[email protected]>
Gerrit-Reviewer: Saggi Mizrahi <[email protected]>
Gerrit-Reviewer: Sergey Gotliv <[email protected]>
Gerrit-Reviewer: Vered Volansky <[email protected]>
Gerrit-Reviewer: Yeela Kaplan <[email protected]>
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes
_______________________________________________
vdsm-patches mailing list
[email protected]
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
