Sahina Bose has posted comments on this change.

Change subject: gluster: Handle missing glusterfs-cli package
......................................................................


Patch Set 6:

(1 comment)

Sorry - i'd thought i had commented on this - but it was stuck in drafts!

https://gerrit.ovirt.org/#/c/50363/6//COMMIT_MSG
Commit Message:

Line 14: mount
isn't it to automatically detect backup servers to pass as mount options.
In case the user has provided this in the mount options manually, this would 
still work without glusterfs-cli IMO. Maybe the warning and commit message 
needs to reflect that?


-- 
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

Reply via email to