[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 08/14] qapi: fix example of FAILOVER_NEGOTIATED event
From: |
John Snow |
Subject: |
Re: [PATCH 08/14] qapi: fix example of FAILOVER_NEGOTIATED event |
Date: |
Thu, 24 Mar 2022 17:05:44 -0400 |
On Thu, Mar 24, 2022 at 1:50 PM Victor Toso <victortoso@redhat.com> wrote:
>
> * Data is an object, not a string. It generates a qdict.
> * Timestamp is not optional, let's add for completeness.
>
> Signed-off-by: Victor Toso <victortoso@redhat.com>
> ---
> qapi/net.json | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/qapi/net.json b/qapi/net.json
> index 7fab2e7cd8..82c0d9e778 100644
> --- a/qapi/net.json
> +++ b/qapi/net.json
> @@ -691,7 +691,8 @@
> # Example:
> #
> # <- { "event": "FAILOVER_NEGOTIATED",
> -# "data": "net1" }
> +# "data": { "device-id": "net1" },
> +# "timestamp": { "seconds": 1368697518, "microseconds": 326866 } }
> #
> ##
> { 'event': 'FAILOVER_NEGOTIATED',
> --
> 2.35.1
>
Oh, this one is all messed up to hell. We're not documenting the
device-id properly, either. Can that be fixed as well?
Your patch ain't wrong, though:
Reviewed-by: John Snow <jsnow@redhat.com>
- Re: [PATCH 06/14] qapi: fix example of MEMORY_DEVICE_SIZE_CHANGE event, (continued)
- [PATCH 08/14] qapi: fix example of FAILOVER_NEGOTIATED event, Victor Toso, 2022/03/24
- Re: [PATCH 08/14] qapi: fix example of FAILOVER_NEGOTIATED event,
John Snow <=
- [PATCH 14/14] qapi: fix example of dump-guest-memory, Victor Toso, 2022/03/24
- [PATCH 12/14] qapi: ui examples: add missing websocket member, Victor Toso, 2022/03/24
- [PATCH 02/14] qapi: fix example of BLOCK_IMAGE_CORRUPTED event, Victor Toso, 2022/03/24