Nir Soffer has posted comments on this change. Change subject: gluster: Allow gluster mount with additional servers ......................................................................
Patch Set 20: (1 comment) https://gerrit.ovirt.org/#/c/40665/20//COMMIT_MSG Commit Message: Line 13: gluster 'backup-volfile-servers' option. Line 14: Line 15: Note that today vdsm has soft-dependency glusterfs-fuse and with this Line 16: change, vdsm will have dependency on glusterfs-cli. If this dependency Line 17: doesn't exist, there is an error when creating gluster storage domain. > Can you add an example of how the code change when using vdscli in order to There is not chnage in the api, old call will work without any change. If you were mounting before with volfile-backup-servers=server1:server2 option, you don't have to do this anymore, since vdsm will get the servers addresses from gluster, and add this mount option automatically. Ala, please explain this in the commit message. Line 18: Line 19: Change-Id: I2478a5edc1fc9d24eb96d64a32a98a2467ce2989 Line 20: Bug-Url: https://bugzilla.redhat.com/1177777 -- To view, visit https://gerrit.ovirt.org/40665 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I2478a5edc1fc9d24eb96d64a32a98a2467ce2989 Gerrit-PatchSet: 20 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Ala Hino <[email protected]> Gerrit-Reviewer: Adam Litke <[email protected]> Gerrit-Reviewer: Ala Hino <[email protected]> Gerrit-Reviewer: Allon Mureinik <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Darshan N <[email protected]> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Nir Soffer <[email protected]> Gerrit-Reviewer: Sahina Bose <[email protected]> Gerrit-Reviewer: Sandro Bonazzola <[email protected]> Gerrit-Reviewer: [email protected] Gerrit-HasComments: Yes _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
