From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from smtp3.mail.ru (smtp3.mail.ru [94.100.179.58]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by dev.tarantool.org (Postfix) with ESMTPS id 76CDF445320 for ; Tue, 14 Jul 2020 00:55:08 +0300 (MSK) From: Vladislav Shpilevoy Date: Mon, 13 Jul 2020 23:55:06 +0200 Message-Id: <8f31a7ee83380c5a895d3a4b50b26393d18b9177.1594677287.git.v.shpilevoy@tarantool.org> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Subject: [Tarantool-patches] [PATCH 1/1] test: fix flaky qsync_with_anon.test.lua List-Id: Tarantool development patches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: tarantool-patches@dev.tarantool.org, avtikhon@tarantool.org The test used very small timeout, 0.1 second, to test that a sync transaction works fine. But with too much load on the machine this may appear not enough. Now in the cases where the timeout wasn't supposed to explode the value 1000 seconds is used, more than enough. Closes #5165 --- Branch: http://github.com/tarantool/tarantool/tree/gerold103/gh-5165-qsync_with_anon-flaky Issue: https://github.com/tarantool/tarantool/issues/5165 test/replication/qsync_with_anon.result | 4 ++-- test/replication/qsync_with_anon.test.lua | 4 ++-- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/test/replication/qsync_with_anon.result b/test/replication/qsync_with_anon.result index 59506a608..51f02bcdb 100644 --- a/test/replication/qsync_with_anon.result +++ b/test/replication/qsync_with_anon.result @@ -48,7 +48,7 @@ test_run:switch('default') | --- | - true | ... -box.cfg{replication_synchro_quorum=NUM_INSTANCES, replication_synchro_timeout=0.1} +box.cfg{replication_synchro_quorum=NUM_INSTANCES, replication_synchro_timeout=1000} | --- | ... _ = box.schema.space.create('sync', {is_sync=true, engine=engine}) @@ -126,7 +126,7 @@ test_run:switch('default') | --- | - true | ... -box.cfg{replication_synchro_quorum=NUM_INSTANCES} +box.cfg{replication_synchro_quorum=NUM_INSTANCES, replication_synchro_timeout=1000} | --- | ... box.space.sync:insert{1} -- success diff --git a/test/replication/qsync_with_anon.test.lua b/test/replication/qsync_with_anon.test.lua index aed62775e..5bc7c8be4 100644 --- a/test/replication/qsync_with_anon.test.lua +++ b/test/replication/qsync_with_anon.test.lua @@ -19,7 +19,7 @@ test_run:cmd('switch replica_anon') -- replica. -- Testcase setup. test_run:switch('default') -box.cfg{replication_synchro_quorum=NUM_INSTANCES, replication_synchro_timeout=0.1} +box.cfg{replication_synchro_quorum=NUM_INSTANCES, replication_synchro_timeout=1000} _ = box.schema.space.create('sync', {is_sync=true, engine=engine}) _ = box.space.sync:create_index('pk') -- Testcase body. @@ -45,7 +45,7 @@ box.space.sync:insert{1} -- failure test_run:cmd('switch replica_anon') box.space.sync:select{} -- none test_run:switch('default') -box.cfg{replication_synchro_quorum=NUM_INSTANCES} +box.cfg{replication_synchro_quorum=NUM_INSTANCES, replication_synchro_timeout=1000} box.space.sync:insert{1} -- success test_run:cmd('switch replica_anon') box.space.sync:select{} -- 1 -- 2.21.1 (Apple Git-122.3)