On Wed, Jul 20, 2016 at 03:24:41PM -0600, Eric Blake wrote:
> On 07/19/2016 06:21 PM, Jeff Cody wrote:
> > From: Prasanna Kumar Kalever <prasanna.kale...@redhat.com>
> >
> > this patch adds 'GlusterServer' related schema in qapi/block-core.json
> >
> > [Jeff: minor fix-ups of comments and formatting, per patch reviews]
> >
> > Signed-off-by: Prasanna Kumar Kalever <prasanna.kale...@redhat.com>
> > Reviewed-by: Markus Armbruster <arm...@redhat.com>
> > Reviewed-by: Eric Blake <ebl...@redhat.com>
> > Message-id: 1468947453-5433-5-git-send-email-prasanna.kale...@redhat.com
> > Signed-off-by: Jeff Cody <jc...@redhat.com>
> > ---
>
> > +++ b/qapi/block-core.json
> > @@ -1666,13 +1666,14 @@
>
> > +##
> > +# @BlockdevOptionsGluster
> > +#
> > +# Driver specific block device options for Gluster
> > +#
> > +# @volume: name of gluster volume where VM image resides
> > +#
> > +# @path: absolute path to image file in gluster volume
> > +#
> > +# @server: gluster server description
> > +#
> > +# @debug-level: #optional libgfapi log level (default '4' which is Error)
>
> You got one,
>
> > +#
> > +# Since: 2.7
> > +##
> > +{ 'struct': 'BlockdevOptionsGluster',
> > + 'data': { 'volume': 'str',
> > + 'path': 'str',
> > + 'server': 'GlusterServer',
> > + '*debug_level': 'int' } }
>
> but not the other instance. Can be fixed in a followup patch if the pull
> request has already landed.
>
Oops, sorry Eric, missed that fix-up when applying. Therein lies the danger
of maintainer-applied fixes.
Yes, I believe these are now in master, so I'll a send patch to fix it for
-rc1. Thanks for catching it.
-Jeff
