Nir Soffer has posted comments on this change. Change subject: gluster: Handle missing glusterfs-cli package ......................................................................
Patch Set 6: (1 comment) https://gerrit.ovirt.org/#/c/50363/6//COMMIT_MSG Commit Message: Line 10: required in order to get gluster volume info needed to get gluster Line 11: backup servers and replica related logic. However, during upgrade, if Line 12: glusterfs-cli is not installed, mount failed. In this case, we'd like to Line 13: add a warning message clearly indicating about the missing glustefs-cli Line 14: package, ask the admin to install it in order to mount backup servers, > isn't it to automatically detect backup servers to pass as mount options. Agree, the warning is relevant only when trying to get volume info. You can change the warning so it warns about automatic detection of back servers. Line 15: and mount gluster server without backup servers. Line 16: Line 17: Change-Id: I9c33aa33e4ffe6a382d40e1bc63f6735efcfcd1f Line 18: Signed-off-by: Ala Hino <ah...@redhat.com> -- To view, visit https://gerrit.ovirt.org/50363 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I9c33aa33e4ffe6a382d40e1bc63f6735efcfcd1f Gerrit-PatchSet: 6 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Ala Hino <ah...@redhat.com> Gerrit-Reviewer: Adam Litke <ali...@redhat.com> Gerrit-Reviewer: Ala Hino <ah...@redhat.com> Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Nir Soffer <nsof...@redhat.com> Gerrit-Reviewer: Sahina Bose <sab...@redhat.com> Gerrit-Reviewer: gerrit-hooks <automat...@ovirt.org> Gerrit-HasComments: Yes _______________________________________________ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches