Re: [libvirt] [PATCH v2] Add support for Veritas HyperScale (VxHS) block device protocol

On Mon, Jan 16, 2017 at 1:10 AM, Daniel P. Berrange <berrange(a)redhat.com&gt; wrote: > On Sat, Jan 14, 2017 at 10:31:05AM -0500, John Ferlan wrote:

> The pre-existing drivers in libvirt need to support both new & old syntax, > in order that we have compat with older QEMU versions. As a brand new driver, > IMHO, VXHS must support the new syntax exclusively - there's no reason to > use the old syntax. We don't need to convert all existing drivers to the > new syntax at the same time, but the VHXS patch can start the trend. > Sometime back, during the qemu patch discussion, we had agreed to have support for both URI syntax and also the QAPI syntax within qemu. It was also decided to limit the URI syntax to supporting only one host server, and use the QAPI exclusively for more than one servers. Now that the failover code has moved from qemu to libqnio, vxhs will accept only one server. Failover will be handled entirely within libqnio. Given that gluster is the only network protocol that supports the new syntax, and that too only for the case when more than one servers are specified, would it be acceptable if vxhs goes with the URI syntax for now? We could add support for the new syntax to vxhs later when doing it for the other protocols!