Dan Kenigsberg has submitted this change and it was merged.
Change subject: glusterHostsList returns host info as list of dictionaries ......................................................................
glusterHostsList returns host info as list of dictionaries
Return value looks like {'status': {'code': CODE, 'message': MESSAGE}, 'hosts' : [{'hostname': HOSTNAME, 'uuid': UUID, 'status': STATE}, ...]}
As this API is not consumed by engine/RHS-C yet, its OK to differ in compatibility issue now.
Change-Id: Ia05d59020bd4019dab2020988dc621d026b805b6 Signed-off-by: Bala.FA barumuga@redhat.com --- M tests/gluster_cli_tests.py M vdsm/gluster/api.py M vdsm/gluster/cli.py 3 files changed, 19 insertions(+), 23 deletions(-)
Approvals: Timothy Asir: Verified; 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/6614 To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged Gerrit-Change-Id: Ia05d59020bd4019dab2020988dc621d026b805b6 Gerrit-PatchSet: 2 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Bala.FA barumuga@redhat.com Gerrit-Reviewer: Ayal Baron abaron@redhat.com Gerrit-Reviewer: Bala.FA barumuga@redhat.com Gerrit-Reviewer: Dan Kenigsberg danken@redhat.com Gerrit-Reviewer: Federico Simoncelli fsimonce@redhat.com Gerrit-Reviewer: Saggi Mizrahi smizrahi@redhat.com Gerrit-Reviewer: Timothy Asir tjeyasin@redhat.com Gerrit-Reviewer: oVirt Jenkins CI Server