[pve-devel] [PATCH qemu-server] fix #2697: map netdev_add options to correct json types
Dominik Csapak
d.csapak at proxmox.com
Tue Apr 21 16:43:12 CEST 2020
On 4/21/20 4:38 PM, Stefan Reiter wrote:
> On 21/04/2020 16:01, Dominik Csapak wrote:
>> netdev_add is now a proper qmp command, which means that it verifies
>> the parameter types properly
>>
>
> The patch looks reasonable, but I'm confused how that worked before?
> 'git blame' tells me the typed QMP schema for netdev_add has been in
> QEMU since 2017..
>
yes but it changed recently (17. March 2020)
its commit:
db2a380c84574d8c76d7193b8af8535234fe5156
net: Complete qapi-fication of netdev_add
it changes netdev_add from
{ 'command': 'netdev_add',
'data': {'type': 'str', 'id': 'str'},
'gen': false } # so we can get the additional arguments
to
{ 'command': 'netdev_add', 'data': 'Netdev', 'boxed': true }
More information about the pve-devel
mailing list