Change in vdsm[master]: Offset by one for UUID position

fsimonce at redhat.com fsimonce at redhat.com
Wed Aug 15 18:13:53 UTC 2012


Federico Simoncelli has posted comments on this change.

Change subject: Offset by one for UUID position
......................................................................


Patch Set 1: (1 inline comment)

....................................................
Commit Message
Line 5: CommitDate: 2012-08-10 18:34:59 +0800
Line 6: 
Line 7: Offset by one for UUID position
Line 8: 
Line 9: In the "metaList" list, "data-center" should be in position 2 instead of 3 which is the result of len(os.path.normpath(config.get('irs', 'repository')).split('/')).  The storage domain uuid should be in metaList[5] instead of metaList[6].  So offset by one here to get the right uuid for the storage domain.
Please cut the lines to the default 72 characters.
Line 10: 
Line 11: Change-Id: I9931a1aa804d9dc7aec1485ebc29bac45131d0e7


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I9931a1aa804d9dc7aec1485ebc29bac45131d0e7
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Shu Ming <shuming at linux.vnet.ibm.com>
Gerrit-Reviewer: Ayal Baron <abaron at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Douglas Schilling Landgraf <dougsland at redhat.com>
Gerrit-Reviewer: Eduardo <ewarszaw at redhat.com>
Gerrit-Reviewer: Federico Simoncelli <fsimonce at redhat.com>
Gerrit-Reviewer: Shu Ming <shuming at linux.vnet.ibm.com>
Gerrit-Reviewer: oVirt Jenkins CI Server


More information about the vdsm-patches mailing list