Francesco Romani 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 <[email protected]> Reviewed-on: https://gerrit.ovirt.org/45098 Reviewed-by: Nir Soffer <[email protected]> Reviewed-by: Bala.FA <[email protected]> Continuous-Integration: Jenkins CI Reviewed-on: https://gerrit.ovirt.org/45370 Reviewed-by: Francesco Romani <[email protected]> --- 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, but someone else must approve Jenkins CI: Passed CI tests Francesco Romani: Looks good to me, approved Ala Hino: Verified -- To view, visit https://gerrit.ovirt.org/45370 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib9c70e9afa2fd776dfd9c2767c2e83e7f179b5c2 Gerrit-PatchSet: 2 Gerrit-Project: vdsm Gerrit-Branch: ovirt-3.6 Gerrit-Owner: Ala Hino <[email protected]> Gerrit-Reviewer: Ala Hino <[email protected]> Gerrit-Reviewer: Francesco Romani <[email protected]> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Nir Soffer <[email protected]> Gerrit-Reviewer: [email protected] _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
