Change in vdsm[master]: gluster: Publish gluster get volume info

danken at redhat.com danken at redhat.com
Wed Aug 26 14:10:00 UTC 2015


Dan Kenigsberg has submitted this change and it was merged.

Change subject: gluster: Publish gluster get volume info
......................................................................


gluster: Publish gluster get volume info

In 3.6 we use glusterVolumeInfo API to get gluster volume info in order to
mount gluster to backup volume servers.  glusterVolumeInfo API is defined in
vdsm/gluster/cli.py and is imported to supervdsm. In vdsm/gluster/cli.py there
are APIs that are used to manage gluster volumes; however, we don't want to
expose all these APIs in supervdsm unless gluster is used, while we do want to
expose getGlusterVolumeInfo.  To achieve this goal, we expose RHEV_ENABLED
param. With this change, gluster API defined in vdsm/gluster/cli.py exposed on
supervdsm either when building with gluster or when building on RHEV.

Change-Id: Ib9c70e9afa2fd776dfd9c2767c2e83e7f179b5c2
Bug-Url: https://bugzilla.redhat.com/1254499
Signed-off-by: Ala Hino <ahino at redhat.com>
Reviewed-on: https://gerrit.ovirt.org/45098
Reviewed-by: Nir Soffer <nsoffer at redhat.com>
Reviewed-by: Bala.FA <barumuga at redhat.com>
Continuous-Integration: Jenkins CI
---
M lib/vdsm/constants.py.in
M vdsm.spec.in
M vdsm/gluster/Makefile.am
M vdsm/gluster/__init__.py
M vdsm/gluster/cli.py
M vdsm/supervdsmServer
6 files changed, 30 insertions(+), 12 deletions(-)

Approvals:
  Nir Soffer: Looks good to me, approved
  Bala.FA: Looks good to me, but someone else must approve
  Jenkins CI: Passed CI tests
  Ala Hino: Verified



-- 
To view, visit https://gerrit.ovirt.org/45098
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ib9c70e9afa2fd776dfd9c2767c2e83e7f179b5c2
Gerrit-PatchSet: 13
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Ala Hino <ahino at redhat.com>
Gerrit-Reviewer: Ala Hino <ahino at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Bala.FA <barumuga at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Darshan N <dnarayan at redhat.com>
Gerrit-Reviewer: Darshan Narayana Murthy <dnarayan at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Nir Soffer <nsoffer at redhat.com>
Gerrit-Reviewer: Sahina Bose <sabose at redhat.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei at redhat.com>
Gerrit-Reviewer: automation at ovirt.org


More information about the vdsm-patches mailing list