Change in vdsm[master]: gluster: additional parsing of host UUID in verb glusterVolu...
dnarayan at redhat.com
dnarayan at redhat.com
Mon Jan 20 06:13:35 UTC 2014
Darshan N has posted comments on this change.
Change subject: gluster: additional parsing of host UUID in verb glusterVolumesList
......................................................................
Patch Set 5:
(1 comment)
http://gerrit.ovirt.org/#/c/22693/5/vdsm/gluster/cli.py
File vdsm/gluster/cli.py:
Line 367: try:
Line 368: brickDetail['name'] = d.find('name').text
Line 369: brickDetail['hostUuid'] = d.find('hostUuid').text
Line 370: value['bricksInfo'].append(brickDetail)
Line 371: except:
> 1. Please catch exact exception.
Done
Line 372: pass
Line 373: volumes[value['volumeName']] = value
Line 374: return volumes
Line 375:
--
To view, visit http://gerrit.ovirt.org/22693
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I9057f3aea0c0ed8fb4d4ec26eaf66fe80ec581e2
Gerrit-PatchSet: 5
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Darshan N <dnarayan at redhat.com>
Gerrit-Reviewer: Aravinda VK <avishwan at redhat.com>
Gerrit-Reviewer: Bala.FA <barumuga at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Darshan N <dnarayan at redhat.com>
Gerrit-Reviewer: Kanagaraj M <kmayilsa at redhat.com>
Gerrit-Reviewer: Sahina Bose <sabose at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes
More information about the vdsm-patches
mailing list