Yaniv Bronhaim has submitted this change and it was merged.

Change subject: gluster: adopt xml changes of rebalance and remove-brick status
......................................................................


gluster: adopt xml changes of rebalance and remove-brick status

xml output of rebalance and remove-brick status add host uuid and task
runtime (per host and summary).  This patch adopts these additional
elements below verbs

* glusterVolumeRebalanceStatus
* glusterVolumeRemoveBrickStatus

Below is the change in output which doesn't break backward
compatibility

        {'summary': {
    ==>>    'runtime': FLOAT,
            'filesScanned': INT,
            'filesMoved': INT,
            'filesFailed': INT,
            'filesSkipped': INT,
            'totalSizeMoved': INT,
            'status': STRING
        },
        'hosts': [{
            'name': STRING,
    ==>>    'id': STRING,
    ==>>    'runtime': FLOAT,
            'filesScanned': INT,
            'filesMoved': INT,
            'filesFailed': INT,
            'filesSkipped': INT,
            'totalSizeMoved': INT,
            'status': STRING
        }]}

Change-Id: I5866ff338768bd5162acc313eed132526b7777ef
Reviewed-on: http://gerrit.ovirt.org/19597
Reviewed-by: Timothy Asir <[email protected]>
Reviewed-by: Dan Kenigsberg <[email protected]>
Signed-off-by: Bala.FA <[email protected]>
Reviewed-on: http://gerrit.ovirt.org/23649
---
M tests/glusterTestData.py
M tests/glusterVolumeRebalanceStatus.xml
M tests/glusterVolumeRemoveBricksStatus.xml
M vdsm/gluster/cli.py
4 files changed, 22 insertions(+), 7 deletions(-)

Approvals:
  Bala.FA: Verified
  Dan Kenigsberg: Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I5866ff338768bd5162acc313eed132526b7777ef
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-3.4
Gerrit-Owner: Bala.FA <[email protected]>
Gerrit-Reviewer: Aravinda VK <[email protected]>
Gerrit-Reviewer: Bala.FA <[email protected]>
Gerrit-Reviewer: Dan Kenigsberg <[email protected]>
Gerrit-Reviewer: Darshan N <[email protected]>
Gerrit-Reviewer: Saggi Mizrahi <[email protected]>
Gerrit-Reviewer: Timothy Asir <[email protected]>
Gerrit-Reviewer: Yaniv Bronhaim <[email protected]>
_______________________________________________
vdsm-patches mailing list
[email protected]
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to