| Message ID | 20220401110712.26911-1-victortoso@redhat.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | [v1.1,1/9] qapi: fix example of netdev_add command | expand |
Victor Toso <victortoso@redhat.com> writes: > Example output has the optional member @dnssearch as string type. It > should be an array of strings instead. Fix it. "of String objects". Happy to fix this in my tree. > > For reference, see NetdevUserOptions. > > Signed-off-by: Victor Toso <victortoso@redhat.com> > --- > qapi/net.json | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/qapi/net.json b/qapi/net.json > index 0d4578bd07..b92f3f5fb4 100644 > --- a/qapi/net.json > +++ b/qapi/net.json > @@ -51,7 +51,7 @@ > # > # -> { "execute": "netdev_add", > # "arguments": { "type": "user", "id": "netdev1", > -# "dnssearch": "example.org" } } > +# "dnssearch": [ { "str": "example.org" } ] } } > # <- { "return": {} } > # > ## Preferably with the commit message tweak: Reviewed-by: Markus Armbruster <armbru@redhat.com>
Hi, On Mon, Apr 04, 2022 at 08:14:11AM +0200, Markus Armbruster wrote: > Victor Toso <victortoso@redhat.com> writes: > > > Example output has the optional member @dnssearch as string type. It > > should be an array of strings instead. Fix it. > > "of String objects". Happy to fix this in my tree. Sure > > > > > For reference, see NetdevUserOptions. > > > > Signed-off-by: Victor Toso <victortoso@redhat.com> > > --- > > qapi/net.json | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/qapi/net.json b/qapi/net.json > > index 0d4578bd07..b92f3f5fb4 100644 > > --- a/qapi/net.json > > +++ b/qapi/net.json > > @@ -51,7 +51,7 @@ > > # > > # -> { "execute": "netdev_add", > > # "arguments": { "type": "user", "id": "netdev1", > > -# "dnssearch": "example.org" } } > > +# "dnssearch": [ { "str": "example.org" } ] } } > > # <- { "return": {} } > > # > > ## > > Preferably with the commit message tweak: > Reviewed-by: Markus Armbruster <armbru@redhat.com> Thanks, Victor
diff --git a/qapi/net.json b/qapi/net.json index 0d4578bd07..b92f3f5fb4 100644 --- a/qapi/net.json +++ b/qapi/net.json @@ -51,7 +51,7 @@ # # -> { "execute": "netdev_add", # "arguments": { "type": "user", "id": "netdev1", -# "dnssearch": "example.org" } } +# "dnssearch": [ { "str": "example.org" } ] } } # <- { "return": {} } # ##
Example output has the optional member @dnssearch as string type. It should be an array of strings instead. Fix it. For reference, see NetdevUserOptions. Signed-off-by: Victor Toso <victortoso@redhat.com> --- qapi/net.json | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)