[Tarantool-patches] [PATCH v1] test: flaky replication/gh-3704-misc-*
Serge Petrenko
sergepetrenko at tarantool.org
Mon Sep 14 10:35:07 MSK 2020
11.09.2020 11:56, Alexander V. Tikhonov пишет:
> On heavy loaded hosts found the following issue:
>
> [037] --- replication/gh-3704-misc-replica-checks-cluster-id.result Thu Sep 10 18:05:22 2020
> [037] +++ replication/gh-3704-misc-replica-checks-cluster-id.reject Fri Sep 11 11:09:38 2020
> [037] @@ -25,7 +25,7 @@
> [037] ...
> [037] box.info.replication[2].downstream.status
> [037] ---
> [037] -- follow
> [037] +- stopped
> [037] ...
> [037] -- change master's cluster uuid and check that replica doesn't connect.
> [037] test_run:cmd("stop server replica")
>
> It happened because replication downstream status check occurred too
> early, when it was only in 'stopped' state. To give the replication
> status check routine ability to reach the needed 'follow' state, it
> need to wait for it using test_run:wait_downstream() routine.
>
> Closes #5293
> ---
Thanks for the patch! LGTM.
>
> Github: https://github.com/tarantool/tarantool/tree/avtikhon/gh-5293-fix-3704
> Issue: https://github.com/tarantool/tarantool/issues/5293
>
> .../replication/gh-3704-misc-replica-checks-cluster-id.result | 4 ++--
> .../gh-3704-misc-replica-checks-cluster-id.test.lua | 2 +-
> 2 files changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/test/replication/gh-3704-misc-replica-checks-cluster-id.result b/test/replication/gh-3704-misc-replica-checks-cluster-id.result
> index 1ca2913f8..27b4393ff 100644
> --- a/test/replication/gh-3704-misc-replica-checks-cluster-id.result
> +++ b/test/replication/gh-3704-misc-replica-checks-cluster-id.result
> @@ -23,9 +23,9 @@ test_run:grep_log("replica", "REPLICASET_UUID_MISMATCH")
> ---
> - null
> ...
> -box.info.replication[2].downstream.status
> +test_run:wait_downstream(2, {status = 'follow'})
> ---
> -- follow
> +- true
> ...
> -- change master's cluster uuid and check that replica doesn't connect.
> test_run:cmd("stop server replica")
> diff --git a/test/replication/gh-3704-misc-replica-checks-cluster-id.test.lua b/test/replication/gh-3704-misc-replica-checks-cluster-id.test.lua
> index 00c443a55..0d5378cae 100644
> --- a/test/replication/gh-3704-misc-replica-checks-cluster-id.test.lua
> +++ b/test/replication/gh-3704-misc-replica-checks-cluster-id.test.lua
> @@ -9,7 +9,7 @@ test_run:cmd("create server replica with rpl_master=default, script='replication
> box.schema.user.grant("guest", "replication")
> test_run:cmd("start server replica")
> test_run:grep_log("replica", "REPLICASET_UUID_MISMATCH")
> -box.info.replication[2].downstream.status
> +test_run:wait_downstream(2, {status = 'follow'})
> -- change master's cluster uuid and check that replica doesn't connect.
> test_run:cmd("stop server replica")
> _ = box.space._schema:replace{'cluster', tostring(uuid.new())}
--
Serge Petrenko
More information about the Tarantool-patches
mailing list