[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v17 0/4][WIP] block/gluster: add support for mul
From: |
Daniel P. Berrange |
Subject: |
Re: [Qemu-devel] [PATCH v17 0/4][WIP] block/gluster: add support for multiple gluster servers |
Date: |
Wed, 29 Jun 2016 15:17:01 +0100 |
User-agent: |
Mutt/1.6.1 (2016-04-27) |
On Wed, Jun 29, 2016 at 10:11:31AM -0400, Jeff Cody wrote:
> On Wed, Jun 15, 2016 at 01:55:43PM +0530, Prasanna Kumar Kalever wrote:
> > This version of patches are rebased on master branch.
> >
> > Prasanna Kumar Kalever (4):
> > block/gluster: rename [server, volname, image] -> [host, volume, path]
> > block/gluster: code cleanup
> > block/gluster: using new qapi schema
> > block/gluster: add support for multiple gluster servers
> >
>
> I think the main criticism with this series revolves around the interface,
> and the overloading of the server hosts fields when using tcp and unix
> sockets, etc. The idea of using flat unions for the API was floated.
>
> Eric, does this criticism still stand, from libvirt's perspective? Or are
> you comfortable enough with the current interface that I can go ahead and
> take this series in through my tree?
Just from a general QAPI design POV I think this overloading is undesirable.
We cared enough about not doing this overloading in the past that we
created SocketAddress which is a union of InetSocketAddress and
UnixSocketAddress. Given this historical best practice, I don't think
we should be overloading "host" for unix socket path.
Regards,
Daniel
--
|: http://berrange.com -o- http://www.flickr.com/photos/dberrange/ :|
|: http://libvirt.org -o- http://virt-manager.org :|
|: http://autobuild.org -o- http://search.cpan.org/~danberr/ :|
|: http://entangle-photo.org -o- http://live.gnome.org/gtk-vnc :|
- [Qemu-devel] [PATCH v17 0/4][WIP] block/gluster: add support for multiple gluster servers, Prasanna Kumar Kalever, 2016/06/15
- [Qemu-devel] [PATCH v17 1/4] block/gluster: rename [server, volname, image] -> [host, volume, path], Prasanna Kumar Kalever, 2016/06/15
- [Qemu-devel] [PATCH v17 2/4] block/gluster: code cleanup, Prasanna Kumar Kalever, 2016/06/15
- [Qemu-devel] [PATCH v17 3/4] block/gluster: using new qapi schema, Prasanna Kumar Kalever, 2016/06/15
- [Qemu-devel] [PATCH v17 4/4] block/gluster: add support for multiple gluster servers, Prasanna Kumar Kalever, 2016/06/15
- Re: [Qemu-devel] [PATCH v17 0/4][WIP] block/gluster: add support for multiple gluster servers, Jeff Cody, 2016/06/29
- Re: [Qemu-devel] [PATCH v17 0/4][WIP] block/gluster: add support for multiple gluster servers,
Daniel P. Berrange <=