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