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