Change in vdsm[master]: gluster: Allow gluster mount with additional servers

nsoffer at redhat.com nsoffer at redhat.com
Tue Jun 16 12:13:30 UTC 2015


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 <ahino at redhat.com>
Gerrit-Reviewer: Adam Litke <alitke at redhat.com>
Gerrit-Reviewer: Ala Hino <ahino at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Darshan N <dnarayan at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Nir Soffer <nsoffer at redhat.com>
Gerrit-Reviewer: Sahina Bose <sabose at redhat.com>
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-HasComments: Yes


More information about the vdsm-patches mailing list