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

Reply via email to