On 05/29/2014 05:14 AM, Chrysostomos Nanakos wrote: > Hello team, > > this is a patch implementing support for a new storage layer, > Archipelago [1][2]. > > We've been using Archipelago in our IaaS public cloud production > environment for over a year now, along with Google Ganeti [3] > and Ceph [4]. We are currently moving from accessing Archipelago > through a kernel block driver to accessing it directly through QEMU, > using this patch.
Are you also planning on extending QMP BlockdevOptions to support this new backend during device hotplug? And if so, it would be nicer if you didn't abuse the file= field, but instead used separate structured fields for each piece of information being passed. We already have the json: pseudo-protocol that can be used to form a flat filename that encodes multiple fields of information without having to further invent new protocols that abuse the purpose of the filename field. -- Eric Blake eblake redhat com +1-919-301-3266 Libvirt virtualization library http://libvirt.org
signature.asc
Description: OpenPGP digital signature