diff mbox series

[V12,19/19] docs: Add COLO status diagram to COLO-FT.txt

Message ID 20180903043900.28592-20-zhangckid@gmail.com (mailing list archive)
State New, archived
Headers show
Series COLO: integrate colo frame with block replication and COLO proxy | expand

Commit Message

Zhang Chen Sept. 3, 2018, 4:39 a.m. UTC
From: Zhang Chen <chen.zhang@intel.com>

This diagram make user better understand COLO.
Suggested by Markus Armbruster.

Signed-off-by: Zhang Chen <zhangckid@gmail.com>
Signed-off-by: Zhang Chen <chen.zhang@intel.com>
---
 docs/COLO-FT.txt | 34 ++++++++++++++++++++++++++++++++++
 1 file changed, 34 insertions(+)

Comments

Jason Wang Sept. 12, 2018, 7:43 a.m. UTC | #1
On 2018年09月03日 12:39, Zhang Chen wrote:
> From: Zhang Chen <chen.zhang@intel.com>
>
> This diagram make user better understand COLO.
> Suggested by Markus Armbruster.
>
> Signed-off-by: Zhang Chen <zhangckid@gmail.com>
> Signed-off-by: Zhang Chen <chen.zhang@intel.com>
> ---
>   docs/COLO-FT.txt | 34 ++++++++++++++++++++++++++++++++++
>   1 file changed, 34 insertions(+)
>
> diff --git a/docs/COLO-FT.txt b/docs/COLO-FT.txt
> index d7c7dcda8f..d5007895d1 100644
> --- a/docs/COLO-FT.txt
> +++ b/docs/COLO-FT.txt
> @@ -110,6 +110,40 @@ Note:
>   HeartBeat has not been implemented yet, so you need to trigger failover process
>   by using 'x-colo-lost-heartbeat' command.
>   
> +== COLO operation status ==
> +
> ++-----------------+
> +|                 |
> +|    Start COLO   |
> +|                 |
> ++--------+--------+
> +         |
> +         |  Main qmp command:
> +         |  migrate-set-capabilities with x-colo
> +         |  migrate
> +         |
> +         v
> ++--------+--------+
> +|                 |
> +|  COLO running   |
> +|                 |
> ++--------+--------+
> +         |
> +         |  Main qmp command:
> +         |  x-colo-lost-heartbeat
> +         |  or
> +         |  some error happened
> +         v
> ++--------+--------+
> +|                 |  send qmp event:
> +|  COLO failover  |  COLO_EXIT
> +|                 |
> ++-----------------+
> +
> +COLO use the qmp command switching and report operation status.

It looks to me that "to switch" is better.

> +The diagram just write the main qmp command, you can get the detail
> +in test procedure.

s/write/shows/

I will fix this patch directly.

Thanks

> +
>   == Test procedure ==
>   1. Startup qemu
>   Primary:
diff mbox series

Patch

diff --git a/docs/COLO-FT.txt b/docs/COLO-FT.txt
index d7c7dcda8f..d5007895d1 100644
--- a/docs/COLO-FT.txt
+++ b/docs/COLO-FT.txt
@@ -110,6 +110,40 @@  Note:
 HeartBeat has not been implemented yet, so you need to trigger failover process
 by using 'x-colo-lost-heartbeat' command.
 
+== COLO operation status ==
+
++-----------------+
+|                 |
+|    Start COLO   |
+|                 |
++--------+--------+
+         |
+         |  Main qmp command:
+         |  migrate-set-capabilities with x-colo
+         |  migrate
+         |
+         v
++--------+--------+
+|                 |
+|  COLO running   |
+|                 |
++--------+--------+
+         |
+         |  Main qmp command:
+         |  x-colo-lost-heartbeat
+         |  or
+         |  some error happened
+         v
++--------+--------+
+|                 |  send qmp event:
+|  COLO failover  |  COLO_EXIT
+|                 |
++-----------------+
+
+COLO use the qmp command switching and report operation status.
+The diagram just write the main qmp command, you can get the detail
+in test procedure.
+
 == Test procedure ==
 1. Startup qemu
 Primary: