Re: Bug: object-add qmp command that used to work fails now

[Lukas Straub <lukasstraub2@web.de>]

[-- Attachment #1: Type: text/plain, Size: 1416 bytes --]

On Wed, 31 Mar 2021 15:16:18 -0500
Eric Blake <eblake@redhat.com> wrote:

> On 3/31/21 3:00 PM, Lukas Straub wrote:
> > Hi,
> > This qmp command that used to work:
> > {"execute": "object-add", "arguments": {"qom-type": "filter-mirror", "id": "m0", "props": {"netdev": "hn0", "queue": "tx", "outdev": "mirror0", "vnet_hdr_support": false}}}
> > now fails with the following error:
> > {"error": {"class": "GenericError", "desc": "Parameter 'netdev' is missing"}}
> > 
> > I haven't looked closely into it, but I think
> > 9151e59a8b6e854eb733553c6772351049ca6ab6 "qapi/qom: QAPIfy object-add"
> > is the culprint.  
> 
> Try getting rid of the nesting caused by props:
> 
> {"execute": "object-add", "arguments": {"qom-type": "filter-mirror",
> "id": "m0", "netdev": "hn0", "queue": "tx", "outdev": "mirror0",
> "vnet_hdr_support": false}}

Okay, that works.

> Per docs/system/removed-features.rst, the change was intentional:
> 
> > ``object-add`` option ``props`` (removed in 6.0)
> > ''''''''''''''''''''''''''''''''''''''''''''''''
> > 
> > Specify the properties for the object as top-level arguments instead.  
> 
> although it is unfortunate that you didn't notice the change during the
> deprecation cycle.
>

Yeah, no big deal. I wasn't sure if this is intentional as the change isn't
documented on https://wiki.qemu.org/ChangeLog/6.0 

Regards,
Lukas Straub

-- 


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 833 bytes --]