Change in vdsm[master]: gluster: Fix output format compatibility of removeBrickStatus

avishwan at redhat.com avishwan at redhat.com
Fri Oct 11 04:57:26 UTC 2013


Aravinda VK has uploaded a new change for review.

Change subject: gluster: Fix output format compatibility of removeBrickStatus
......................................................................

gluster: Fix output format compatibility of removeBrickStatus

Rebalance status and Remove brick status have common output structure,
engine uses common code to parse the vdsm output of both verbs.

This patch fixes the incompatibility in remove brick status output.
(As this verb is not consumed by engine/RHS-C yet, its OK to differ
in compatibility now)

Output:
    {'summary': {
        'filesScanned': INT,
        'filesMoved': INT,
        'filesFailed': INT,
        'filesSkipped': INT,
        'totalSizeMoved': INT,
        'status': STRING
    },
    'hosts': [{
        'name': STRING,
        'filesScanned': INT,
        'filesMoved': INT,
        'filesFailed': INT,
        'filesSkipped': INT,
        'totalSizeMoved': INT,
        'status': STRING
    }..]}

Change-Id: Id7996af2a779119a6f703f309ad377c8a425b1c5
Signed-off-by: Aravinda VK <avishwan at redhat.com>
---
M vdsm/gluster/api.py
1 file changed, 3 insertions(+), 4 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/08/20108/1

diff --git a/vdsm/gluster/api.py b/vdsm/gluster/api.py
index ebb0901..284fa97 100644
--- a/vdsm/gluster/api.py
+++ b/vdsm/gluster/api.py
@@ -181,10 +181,9 @@
     @exportAsVerb
     def volumeRemoveBrickStatus(self, volumeName, brickList,
                                 replicaCount=0, options=None):
-        message = self.svdsmProxy.glusterVolumeRemoveBrickStatus(volumeName,
-                                                                 brickList,
-                                                                 replicaCount)
-        return {'message': message}
+        return self.svdsmProxy.glusterVolumeRemoveBrickStatus(volumeName,
+                                                              brickList,
+                                                              replicaCount)
 
     @exportAsVerb
     def volumeRemoveBrickCommit(self, volumeName, brickList,


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: Id7996af2a779119a6f703f309ad377c8a425b1c5
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Aravinda VK <avishwan at redhat.com>


More information about the vdsm-patches mailing list