Change in vdsm[master]: glusterHostsList returns host info as list of dictionaries

danken at redhat.com danken at redhat.com
Wed Aug 15 13:04:34 UTC 2012


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 at 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 at redhat.com>
Gerrit-Reviewer: Ayal Baron <abaron at redhat.com>
Gerrit-Reviewer: Bala.FA <barumuga at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Federico Simoncelli <fsimonce at redhat.com>
Gerrit-Reviewer: Saggi Mizrahi <smizrahi at redhat.com>
Gerrit-Reviewer: Timothy Asir <tjeyasin at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server


More information about the vdsm-patches mailing list