Sahina Bose has posted comments on this change.

Change subject: gluster: geo replication status and status detail
......................................................................


Patch Set 6:

(1 comment)

Darshan, could you address these comments?
Is it possible to make this patch independent of geo-rep create patch?

http://gerrit.ovirt.org/#/c/18414/6/vdsm/gluster/cli.py
File vdsm/gluster/cli.py:

Line 1179:      sessions: [{sessionSlave: 'session slave/uuid of session',
Line 1180:                  pairs: [{masterNode: 'master node',
Line 1181:                          masterNodeUuid: 'uuid of brick host',
Line 1182:                          masterBrick: 'master brick',
Line 1183:                          slave: 'slave',
Is it possible to return the slave host uuid as well, in addition to the slave 
host name?
Line 1184:                          status: 'status'
Line 1185:                          checkpointStatus: 'checkpoint status'
Line 1186:                          crawlStatus: 'crawlStatus'}]....
Line 1187:                ]....


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I4f37f35a5480fbe049a67758e122d4a0c2eba513
Gerrit-PatchSet: 6
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Darshan N <[email protected]>
Gerrit-Reviewer: Aravinda VK <[email protected]>
Gerrit-Reviewer: Bala.FA <[email protected]>
Gerrit-Reviewer: Better Saggi <[email protected]>
Gerrit-Reviewer: Darshan N <[email protected]>
Gerrit-Reviewer: Saggi Mizrahi <[email protected]>
Gerrit-Reviewer: Sahina Bose <[email protected]>
Gerrit-Reviewer: Timothy Asir <[email protected]>
Gerrit-Reviewer: [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

Reply via email to