Change in vdsm[master]: '0' is a valid master version

agl at us.ibm.com agl at us.ibm.com
Wed Sep 12 19:33:28 UTC 2012


Adam Litke has posted comments on this change.

Change subject: '0' is a valid master version
......................................................................


Patch Set 1: I would prefer that you didn't submit this

(1 inline comment)

....................................................
File vdsm/storage/hsm.py
Line 848:                 pool.getMasterDomain(msdUUID=msdUUID, masterVersion=masterVersion)
Line 849:                 return
Line 850: 
Line 851:             pool = sp.StoragePool(spUUID, self.taskMng)
Line 852:             if not hostID or not scsiKey or not msdUUID or (masterVersion == None):
masterVersion is None
Line 853:                 hostID, scsiKey, msdUUID, masterVersion = pool.getPoolParams()
Line 854:             res = pool.connect(hostID, scsiKey, msdUUID, masterVersion)
Line 855:             if res:
Line 856:                 self.pools[spUUID] = pool


--
To view, visit http://gerrit.ovirt.org/7948
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I736d917bfa7d3a5ac59cddcaf1a18846f53d85b0
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Shu Ming <shuming at linux.vnet.ibm.com>
Gerrit-Reviewer: Adam Litke <agl at us.ibm.com>
Gerrit-Reviewer: Eduardo <ewarszaw at redhat.com>
Gerrit-Reviewer: Saggi Mizrahi <smizrahi at redhat.com>


More information about the vdsm-patches mailing list