Dan Kenigsberg 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: I5b3031b6eb796b7475bbe533fbdd0366fbaf6aea
Signed-off-by: Bala.FA <[email protected]>
Reviewed-on: http://gerrit.ovirt.org/19597
Reviewed-by: Timothy Asir <[email protected]>
Reviewed-by: Dan Kenigsberg <[email protected]>
---
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:
  Timothy Asir: Looks good to me, but someone else must approve
  Bala.FA: Verified
  Dan Kenigsberg: Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I5b3031b6eb796b7475bbe533fbdd0366fbaf6aea
Gerrit-PatchSet: 13
Gerrit-Project: vdsm
Gerrit-Branch: master
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: oVirt Jenkins CI Server
_______________________________________________
vdsm-patches mailing list
[email protected]
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to