Sahina Bose has posted comments on this change.
Change subject: virt: enable glusterfs access through libgfapi interface ......................................................................
Patch Set 8:
(1 comment)
https://gerrit.ovirt.org/#/c/44061/8/vdsm/storage/glusterVolume.py File vdsm/storage/glusterVolume.py:
Line 76: for brick in volInfo[volname]['bricks']] Line 77: Line 78: return {'path': glusterPath, Line 79: 'protocol': 'gluster', Line 80: 'volfileServer': volfileServer,
Why do we need this?
yes volfileserver is one of the address of bricks. However volfileserver could be the user preferred server to access the volume. Especially in case that user has setup virtual IP, they may provide that as the volfileserver. We can change to add volfileserver as the first element in hosts, as I see it is not used in prepareVolumePath - let me know if right approach