Re: [Qemu-devel] [PATCH] qapi/migration.json: Clean up for COLOStatus

[Markus Armbruster]

Zhang Chen <address@hidden > writes:

> From: Zhang Chen <address@hidden>
>
> Address Markus's comments change the 'last_mode' to 'last-mode'.
>
> Signed-off-by: Zhang Chen <address@hidden>
> ---
>  qapi/migration.json | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/qapi/migration.json b/qapi/migration.json
> index cfde29acf8..9217886c80 100644
> --- a/qapi/migration.json
> +++ b/qapi/migration.json
> @@ -1380,7 +1380,7 @@
>  # @mode: COLO running mode. If COLO is running, this field will return
>  #        'primary' or 'secondary'.
>  #
> -# @last_mode: COLO last running mode. If COLO is running, this field
> +# @last-mode: COLO last running mode. If COLO is running, this field
>  #             will return same like mode field, after failover we can
>  #             use this field to get last colo mode. (since 4.1)
>  #
> @@ -1389,7 +1389,7 @@
>  # Since: 3.1
>  ##
>  { 'struct': 'COLOStatus',
> -  'data': { 'mode': 'COLOMode', 'last_mode': 'COLOMode',
> +  'data': { 'mode': 'COLOMode', 'last-mode': 'COLOMode',
>              'reason': 'COLOExitReason' } }
>  
>  ##

Reviewed-by: Markus Armbruster <address@hidden>

Queued for 4.0 with the commit message tweaked to just

    qapi/migration.json: Rename COLOStatus last_mode to last-mode

Hope that's okay.  Thank you!