[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v3 11/14] qapi: fix example of MEMORY_FAILURE
From: |
Markus Armbruster |
Subject: |
Re: [PATCH v3 11/14] qapi: fix example of MEMORY_FAILURE |
Date: |
Tue, 29 Mar 2022 17:52:47 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/27.2 (gnu/linux) |
Victor Toso <victortoso@redhat.com> writes:
> Example output lacks mandatory member @timestamp. Provide it.
>
> While @flags is an optional member,
Uh, it isn't.
> if it is defined then all its
> members should be include. For that reason, we add @recursive member.
Perhaps:
Example output lacks mandatory member flags.recursive. Provide it.
Happy to make such a change in my tree.
> Minor: Change quotes from '' to "" in @action-required member.
>
> Signed-off-by: Victor Toso <victortoso@redhat.com>
> Reviewed-by: John Snow <jsnow@redhat.com>
> ---
> qapi/run-state.json | 4 +++-
> 1 file changed, 3 insertions(+), 1 deletion(-)
>
> diff --git a/qapi/run-state.json b/qapi/run-state.json
> index 7f1c788c4e..8124220bd9 100644
> --- a/qapi/run-state.json
> +++ b/qapi/run-state.json
> @@ -571,7 +571,9 @@
> # <- { "event": "MEMORY_FAILURE",
> # "data": { "recipient": "hypervisor",
> # "action": "fatal",
> -# "flags": { 'action-required': false } }
> +# "flags": { "action-required": false,
> +# "recursive": false } },
> +# "timestamp": { "seconds": 1267061043, "microseconds": 959568 } }
> #
> ##
> { 'event': 'MEMORY_FAILURE',
With the commit message fixed:
Reviewed-by: Markus Armbruster <armbru@redhat.com>
- [PATCH v3 07/14] qapi: fix example of UNPLUG_PRIMARY event, (continued)
- [PATCH v3 07/14] qapi: fix example of UNPLUG_PRIMARY event, Victor Toso, 2022/03/28
- [PATCH v3 05/14] qapi: fix example of DUMP_COMPLETED event, Victor Toso, 2022/03/28
- [PATCH v3 08/14] qapi: fix example of FAILOVER_NEGOTIATED event, Victor Toso, 2022/03/28
- [PATCH v3 09/14] qapi: fix examples: SHUTDOWN and RESET events, Victor Toso, 2022/03/28
- [PATCH v3 10/14] qapi: run-state examples: add missing @timestamp, Victor Toso, 2022/03/28
- [PATCH v3 11/14] qapi: fix example of MEMORY_FAILURE, Victor Toso, 2022/03/28
- Re: [PATCH v3 11/14] qapi: fix example of MEMORY_FAILURE,
Markus Armbruster <=
- [PATCH v3 12/14] qapi: ui examples: add missing @websocket member, Victor Toso, 2022/03/28
- [PATCH v3 13/14] qapi: fix example of ACPI_DEVICE_OST event, Victor Toso, 2022/03/28
- [PATCH v3 14/14] qapi: fix example of dump-guest-memory, Victor Toso, 2022/03/28
- Re: [PATCH v3 00/14] Fix some qapi examples and a TODO section, Markus Armbruster, 2022/03/29
- Re: [PATCH v3 00/14] Fix some qapi examples and a TODO section, Markus Armbruster, 2022/03/31