Change in vdsm[master]: storage: Fix the case when VDSM host is not a gluster peer

danken at redhat.com danken at redhat.com
Wed May 15 08:47:39 UTC 2013


Dan Kenigsberg has submitted this change and it was merged.

Change subject: storage: Fix the case when VDSM host is not a gluster peer
......................................................................


storage: Fix the case when VDSM host is not a gluster peer

Currently when using glusterSD, VDSM has the need to know the
transport type of gluster volume for which it uses gluster cli
which assumes that VDSM host is part of gluster cluster (peer).
This may not be true always and imposes a constraint that VDSM
host must be a gluster peer.

This patch fixes the above by doing 2 things...

Adds support for --remote-host gluster cli option to volumeInfo
API. This helps retrieve gluster volume info from a host that
is not a gluster peer (Eg. VDSM host)

Adds support to use the ability of gluster cli (--remote-host) to
retrieve the gluster volume info even when VDSM host is not a
gluster peer.

Change-Id: Ia5967f42555db27c104cd64449fccdc9b5fbc779
Signed-off-by: Deepak C Shetty <deepakcs at linux.vnet.ibm.com>
---
M vdsm/gluster/cli.py
M vdsm/storage/glusterVolume.py
2 files changed, 15 insertions(+), 3 deletions(-)

Approvals:
  Bala.FA: Looks good to me, but someone else must approve
  Dan Kenigsberg: Looks good to me, approved
  Deepak C Shetty: Verified


--
To view, visit http://gerrit.ovirt.org/14666
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ia5967f42555db27c104cd64449fccdc9b5fbc779
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Deepak C Shetty <deepakcs at linux.vnet.ibm.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: Deepak C Shetty <deepakcs at linux.vnet.ibm.com>
Gerrit-Reviewer: Federico Simoncelli <fsimonce at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server


More information about the vdsm-patches mailing list