Re: [libvirt] [PATCH 1/1] qemu: add support for multiple gluster hosts

Am 05.10.2015 um 13:01 hat Peter Krempa geschrieben: > On Mon, Oct 05, 2015 at 15:30:42 +0530, Prasanna Kumar Kalever wrote:

> > resultant string: > > -drive file=json:{ > > > "file": { > > "driver": "gluster",, > > "volname": "testvol",, > > "image-path": "/a.qcow2",, > > "volfile-servers": [ > > { > > "server": "1.2.3.4",, > > "port": 24009,, > > "transport": "tcp" > > },, > > { > > "server": "3.4.5.6",, > > "port": 24008,, > > "transport": "tcp" > > },, > > { > > "server": "5.6.7.8",, > > "port": 24007,, > > The double commas look like a result of our command line escaping > function. Are they actually required with 'json:' sources? If no, we > will need probably a way to avoid them. This looks like it's used on the command line (i.e. the file=... parameter of -drive). In this case, the escaping is necessary so the string isn't split in the middle of the JSON object.

If you used the same thing in a blockdev-add QMP command, you wouldn't need escaping, obviously, and double commas would be a syntax error. It's possible to avoid JSON syntax on the command line, but as you mentioned yourself below, it's necessary in other contexts (backing file strings), so it probably makes sense to use it here as well. For the record, the version without JSON would look like this (without the whitespace; only formatting it this way for readability): -drive file.driver=gluster, file.volname=testvol, file.image-path=/a.qcow2, file.volfile-servers.0.server=1.2.3.4, file.volfile-servers.0.port=24009,