Message ID | 20161206182020.25736-1-eblake@redhat.com (mailing list archive) |
---|---|
State | New, archived |
Headers | show |
On Tue, Dec 06, 2016 at 12:20:20PM -0600, Eric Blake wrote: > We intentionally renamed 'debug-level' to 'debug' in the QMP > schema for 'blockdev-add' related to gluster, in order to > match the command line (commit 1a417e46). However, since > 'debug-level' was visible in 2.7, that means that we should > document that 'debug' was not available until 2.8. > > The change was intentional because 'blockdev-add' itself > underwent incompatible changes (such as commit 0153d2f) for > the same release; our intent is that after 2.8, these > interfaces will now be stable. [In hindsight, we should have > used the name x-blockdev-add when we first introduced it] > > Signed-off-by: Eric Blake <eblake@redhat.com> > --- > qapi/block-core.json | 1 + > 1 file changed, 1 insertion(+) Thanks, applied to my staging tree: https://github.com/stefanha/qemu/commits/staging Stefan
diff --git a/qapi/block-core.json b/qapi/block-core.json index a298e76..6b42216 100644 --- a/qapi/block-core.json +++ b/qapi/block-core.json @@ -2196,6 +2196,7 @@ # @server: gluster servers description # # @debug: #optional libgfapi log level (default '4' which is Error) +# (Since 2.8) # # @logfile: #optional libgfapi log file (default /dev/stderr) (Since 2.8) #
We intentionally renamed 'debug-level' to 'debug' in the QMP schema for 'blockdev-add' related to gluster, in order to match the command line (commit 1a417e46). However, since 'debug-level' was visible in 2.7, that means that we should document that 'debug' was not available until 2.8. The change was intentional because 'blockdev-add' itself underwent incompatible changes (such as commit 0153d2f) for the same release; our intent is that after 2.8, these interfaces will now be stable. [In hindsight, we should have used the name x-blockdev-add when we first introduced it] Signed-off-by: Eric Blake <eblake@redhat.com> --- qapi/block-core.json | 1 + 1 file changed, 1 insertion(+)