Dan Kenigsberg has submitted this change and it was merged. 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 <avish...@redhat.com> Reviewed-on: http://gerrit.ovirt.org/20108 Reviewed-by: Bala.FA <barum...@redhat.com> Reviewed-by: Dan Kenigsberg <dan...@redhat.com> --- M vdsm/gluster/api.py 1 file changed, 3 insertions(+), 4 deletions(-) Approvals: Bala.FA: Looks good to me, but someone else must approve Aravinda VK: Verified Dan Kenigsberg: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/20108 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id7996af2a779119a6f703f309ad377c8a425b1c5 Gerrit-PatchSet: 2 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Aravinda VK <avish...@redhat.com> Gerrit-Reviewer: Aravinda VK <avish...@redhat.com> Gerrit-Reviewer: Bala.FA <barum...@redhat.com> Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com> Gerrit-Reviewer: Timothy Asir <tjeya...@redhat.com> Gerrit-Reviewer: ndarshan <dnara...@redhat.com> _______________________________________________ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches