qemu-trivial
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-trivial] [PATCH] qapi-schema: clarify 'colo' state for Migrati


From: Eric Blake
Subject: Re: [Qemu-trivial] [PATCH] qapi-schema: clarify 'colo' state for MigrationStatus
Date: Tue, 1 Nov 2016 09:28:14 -0500
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.4.0

On 11/01/2016 12:51 AM, zhanghailiang wrote:
> VM can not get into colo state unless users enable 'x-colo'
> capability for migration, Here it is necessary to clarify
> this.
> 
> Suggested-by: Eric Blake <address@hidden>
> Signed-off-by: zhanghailiang <address@hidden>
> ---
>  qapi-schema.json | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/qapi-schema.json b/qapi-schema.json
> index 8a7b527..d580169 100644
> --- a/qapi-schema.json
> +++ b/qapi-schema.json
> @@ -462,7 +462,8 @@
>  #
>  # @failed: some error occurred during migration process.
>  #
> -# @colo: VM is in the process of fault tolerance. (since 2.8)
> +# @colo: VM is in the process of fault tolerance, VM can not get into this
> +#        state unless colo capability is enabled for migration. (since 2.8)

This hunk is for MigrationStatus; can you also add similar wording for
the documentation of RunState?  Looking forward to v2.

-- 
Eric Blake   eblake redhat com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org

Attachment: signature.asc
Description: OpenPGP digital signature


reply via email to

[Prev in Thread] Current Thread [Next in Thread]