Change in vdsm[master]: gluster: adopt xml changes of rebalance and remove-brick status

danken at redhat.com danken at redhat.com
Wed Jan 22 07:42:44 UTC 2014


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 <barumuga at redhat.com>
Reviewed-on: http://gerrit.ovirt.org/19597
Reviewed-by: Timothy Asir <tjeyasin at redhat.com>
Reviewed-by: Dan Kenigsberg <danken at redhat.com>
---
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 <barumuga 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: Saggi Mizrahi <smizrahi at redhat.com>
Gerrit-Reviewer: Timothy Asir <tjeyasin at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server


More information about the vdsm-patches mailing list