Yaniv Bronhaim has submitted this change and it was merged. Change subject: gluster: Handle missing glusterfs-cli package ......................................................................
gluster: Handle missing glusterfs-cli package In 3.6 we introduced a soft dependency to glusterfs-cli. This was required in order to get gluster volume info needed for to purposes: 1. get gluster backup servers in order to automatically mount gluster back servers 2. get replica related to verify that it is supported However, during upgrade, if glusterfs-cli is missing, mount failed. In this case, we'd like to add a warning message clearly indicating about the missing glustefs-cli package, and ask the admin to install it in order to enable automatica mount of gluster backup servers. Change-Id: I9c33aa33e4ffe6a382d40e1bc63f6735efcfcd1f Signed-off-by: Ala Hino <[email protected]> Bug-Url: https://bugzilla.redhat.com/1290997 Reviewed-on: https://gerrit.ovirt.org/50363 Continuous-Integration: Jenkins CI Reviewed-by: Nir Soffer <[email protected]> Reviewed-by: Sahina Bose <[email protected]> Reviewed-on: https://gerrit.ovirt.org/50787 Reviewed-by: Yaniv Bronhaim <[email protected]> --- M tests/storageServerTests.py M vdsm/gluster/cli.py M vdsm/storage/storageServer.py 3 files changed, 33 insertions(+), 2 deletions(-) Approvals: Nir Soffer: Looks good to me, but someone else must approve Yaniv Bronhaim: Looks good to me, approved Jenkins CI: Passed CI tests Ala Hino: Verified -- To view, visit https://gerrit.ovirt.org/50787 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I9c33aa33e4ffe6a382d40e1bc63f6735efcfcd1f 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: Sahina Bose <[email protected]> Gerrit-Reviewer: Yaniv Bronhaim <[email protected]> Gerrit-Reviewer: gerrit-hooks <[email protected]> _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
