Nir Soffer 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/clientIF.py File vdsm/clientIF.py: Line 321: if drive.get('diskType') == DISK_TYPE.NETWORK: Line 322: volinfo = res['info'] Line 323: volPath = volinfo['path'] Line 324: drive['protocol'] = volinfo['protocol'] Line 325: # currently, single host is provided due to this bug: > I think we should take in the support for additional hosts as an addon patc I agree with Sahina, this patch should make it possible to use libgfapi with current support in libvirt. When libvirt will support more than one host we will support multiple hosts. Line 326: # https://bugzilla.redhat.com/1247521 Line 327: drive['hosts'] = [volinfo['hosts'][0]] Line 328: else: Line 329: volPath = res['path'] -- To view, visit https://gerrit.ovirt.org/44061 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I54b81e87b959b0b49c0f06810f88410e7c75de1d Gerrit-PatchSet: 8 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Ala Hino <ah...@redhat.com> Gerrit-Reviewer: Ala Hino <ah...@redhat.com> Gerrit-Reviewer: Allon Mureinik <amure...@redhat.com> Gerrit-Reviewer: Federico Simoncelli <fsimo...@redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Nir Soffer <nsof...@redhat.com> Gerrit-Reviewer: Prasanna Kumar Kalever <prasanna.kale...@redhat.com> Gerrit-Reviewer: Sahina Bose <sab...@redhat.com> Gerrit-Reviewer: Yaniv Kaul <yk...@redhat.com> Gerrit-Reviewer: gerrit-hooks <automat...@ovirt.org> Gerrit-HasComments: Yes _______________________________________________ vdsm-patches mailing list -- vdsm-patches@lists.fedorahosted.org To unsubscribe send an email to vdsm-patches-le...@lists.fedorahosted.org