Aravinda VK has posted comments on this change.

Change subject: gluster: Using XML output for rebalance/remove-brick status 
verbs
......................................................................


Patch Set 2:

(2 comments)

....................................................
File client/vdsClientGluster.py
Line 135: 
Line 136:     def do_glusterVolumeRebalanceStatus(self, args):
Line 137:         params = self._eqSplit(args)
Line 138:         volumeName = params.get('volumeName', '')
Line 139: 
Done
Line 140:         status = self.s.glusterVolumeRebalanceStatus(volumeName)
Line 141:         pp.pprint(status)
Line 142:         return status['status']['code'], status['status']['message']
Line 143: 


....................................................
File vdsm/gluster/cli.py
Line 75: 
Line 76: class TaskType:
Line 77:     REBALANCE = 'REBALANCE'
Line 78:     REPLACE_BRICK = 'REPLACE_BRICK'
Line 79:     REMOVE_BRICK = 'REMOVE_BRICK'
I missed out to move this to tasklist patch. Will copy this to other patch.
Line 80: 
Line 81: 
Line 82: class TaskStatus:
Line 83:     NOT_STARTED = "NOT STARTED"


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I7d277f267cf64d8af419d07367c33dc065f86a9b
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Aravinda VK <avish...@redhat.com>
Gerrit-Reviewer: Aravinda VK <avish...@redhat.com>
Gerrit-Reviewer: Bala.FA <barum...@redhat.com>
Gerrit-Reviewer: Timothy Asir <tjeya...@redhat.com>
Gerrit-Reviewer: ndarshan <dnara...@redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes
_______________________________________________
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to