From: Serge Petrenko <sergepetrenko@tarantool.org> To: Alexander Turenko <alexander.turenko@tarantool.org> Cc: "Alexander V. Tikhonov" <avtikhon@tarantool.org>, tarantool-patches@freelists.org Subject: [tarantool-patches] Re: [PATCH v1] test: replication/misc fix for high load Date: Tue, 11 Jun 2019 10:22:08 +0300 [thread overview] Message-ID: <FA479E71-CDD3-4D20-8D61-E3953A7C7965@tarantool.org> (raw) In-Reply-To: <20190610060756.rhc5nhpfuza72v4e@tkn_work_nb> > 10 июня 2019 г., в 9:07, Alexander Turenko <alexander.turenko@tarantool.org> написал(а): > > I'm okay. > > Serge, do you have any objections? Looks good to me. -- Serge Petrenko sergepetrenko@tarantool.org > > WBR, Alexander Turenko. > > On Fri, Jun 07, 2019 at 04:25:12PM +0300, Alexander V. Tikhonov wrote: >> Fixed test replication/misc to be able to run it on >> hosts under high load. Changed downstream check to >> use test_run:wait_downstream() function, to wait for >> certain box.info.replication values to fix the error: >> >> [050] replication/misc.test.lua [ fail ] >> [050] >> [050] Test failed! Result content mismatch: >> [050] --- replication/misc.result Thu Jun 6 06:46:54 2019 >> [050] +++ replication/misc.reject Fri Jun 7 05:55:00 2019 >> [050] @@ -622,7 +622,7 @@ >> [050] ... >> [050] box.info.replication[2].downstream.status >> [050] --- >> [050] -- stopped >> [050] +- follow >> [050] ... >> [050] test_run:cmd("stop server replica") >> [050] --- >> >> Closes #4277 >> --- >> >> Github: https://github.com/tarantool/tarantool/tree/avtikhon/gh-4277-test-replication-misc >> Issue: https://github.com/tarantool/tarantool/issues/4277 >> >> test/replication/misc.result | 4 ++-- >> test/replication/misc.test.lua | 2 +- >> 2 files changed, 3 insertions(+), 3 deletions(-) >> >> diff --git a/test/replication/misc.result b/test/replication/misc.result >> index 5d7a6512e..f896ee4a9 100644 >> --- a/test/replication/misc.result >> +++ b/test/replication/misc.result >> @@ -620,9 +620,9 @@ test_run:wait_log("replica", "REPLICASET_UUID_MISMATCH", nil, 1.0) >> --- >> - REPLICASET_UUID_MISMATCH >> ... >> -box.info.replication[2].downstream.status >> +test_run:wait_downstream(2, {status = 'stopped'}) >> --- >> -- stopped >> +- true >> ... >> test_run:cmd("stop server replica") >> --- >> diff --git a/test/replication/misc.test.lua b/test/replication/misc.test.lua >> index 5a6cecfaa..4be12ba3b 100644 >> --- a/test/replication/misc.test.lua >> +++ b/test/replication/misc.test.lua >> @@ -251,7 +251,7 @@ _ = box.space._schema:replace{'cluster', tostring(uuid.new())} >> -- master believes replica is in cluster, but their cluster UUIDs differ. >> test_run:cmd("start server replica") >> test_run:wait_log("replica", "REPLICASET_UUID_MISMATCH", nil, 1.0) >> -box.info.replication[2].downstream.status >> +test_run:wait_downstream(2, {status = 'stopped'}) >> >> test_run:cmd("stop server replica") >> test_run:cmd("cleanup server replica") >> -- >> 2.17.1 >>
next prev parent reply other threads:[~2019-06-11 7:22 UTC|newest] Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top 2019-06-07 13:25 [tarantool-patches] " Alexander V. Tikhonov 2019-06-10 6:07 ` [tarantool-patches] " Alexander Turenko 2019-06-11 7:22 ` Serge Petrenko [this message] 2019-06-11 7:37 ` Alexander Turenko 2019-06-13 8:21 ` Kirill Yukhin 2019-06-13 9:50 ` Alexander Turenko
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=FA479E71-CDD3-4D20-8D61-E3953A7C7965@tarantool.org \ --to=sergepetrenko@tarantool.org \ --cc=alexander.turenko@tarantool.org \ --cc=avtikhon@tarantool.org \ --cc=tarantool-patches@freelists.org \ --subject='[tarantool-patches] Re: [PATCH v1] test: replication/misc fix for high load' \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: link
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox