On Tue, Nov 1, 2016 at 11:32 PM, Eric Blake <ebl...@redhat.com> wrote: > On 11/01/2016 07:29 AM, Prasanna Kumar Kalever wrote: >> Document: >> 1. The new debug and logfile options with their usages and >> 2. New json format and its usage. >> >> Signed-off-by: Prasanna Kumar Kalever <prasanna.kale...@redhat.com> >> --- >> qemu-doc.texi | 46 ++++++++++++++++++++++++++++++++-------------- >> qemu-options.hx | 14 ++++++++++++-- >> 2 files changed, 44 insertions(+), 16 deletions(-) >> >> diff --git a/qemu-doc.texi b/qemu-doc.texi >> index 023c140..a7c5722 100644 >> --- a/qemu-doc.texi >> +++ b/qemu-doc.texi >> @@ -1041,35 +1041,50 @@ GlusterFS is an user space distributed file system. >> >> You can boot from the GlusterFS disk image with the command: >> @example >> -qemu-system-x86_64 -drive >> file=gluster[+@var{transport}]://[@var{server}[:@var{port}]]/@var{volname}/@var{image}[?socket=...] >> +URI: >> +qemu-system-x86_64 -drive >> file=gluster[+@var{type}]://[@var{host}[:@var{port}]]/@var{volume}/@var{path}[?socket=...] >> + >> +JSON: >> +qemu-system-x86_64 >> 'json:@{"driver":"qcow2","file":@{"driver":"gluster","volume":"testvol","path":"a.img","debug":"N","logfile":"...","server":[@{"type":"tcp","host":"...","port":"..."@},@{"type":"unix","socket":"..."@}]@}@}' > > "debug":"N" does not match the schema; the parameter is named > "debug-level", and it is an integer not a string. The parameter is > optional; you could just omit it. But if you are going to include it, > give a reasonable example like "debug-level":0.
Eric, >From the code I can see #define GLUSTER_OPT_DEBUG "debug" And even tried this, there is no option 'debug-level'. And yes, I should admit it is 'int' not a 'string' Thanks, rest of the comments seems convincing to me, will make the changes and spin them soon. -- Prasanna [...] >> > > -- > Eric Blake eblake redhat com +1-919-301-3266 > Libvirt virtualization library http://libvirt.org >