Sandro Bonazzola has posted comments on this change.

Change subject: gluster: Allow gluster mount with additional servers
......................................................................


Patch Set 20:

(2 comments)

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 use 
the new mount?
Line 18: 
Line 19: Change-Id: I2478a5edc1fc9d24eb96d64a32a98a2467ce2989
Line 20: Bug-Url: https://bugzilla.redhat.com/1177777


https://gerrit.ovirt.org/#/c/40665/20/vdsm/storage/storageServer.py
File vdsm/storage/storageServer.py:

Line 1: #
2012-2015
Line 2: # Copyright 2012 Red Hat, Inc.
Line 3: #
Line 4: # This program is free software; you can redistribute it and/or modify
Line 5: # it under the terms of the GNU General Public License as published by


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

Reply via email to