On 05/30/2014 12:32 AM, Eric Blake wrote:
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?
Yes of course, that won't be a problem.
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.
Let me make the proposed changes among others and resend a second
version of the proposed patch.
Regards,
Chrysostomos.
