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