From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from smtp61.i.mail.ru (smtp61.i.mail.ru [217.69.128.41]) (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 8D619446439 for ; Thu, 5 Nov 2020 17:35:03 +0300 (MSK) From: "Alexander V. Tikhonov" Date: Thu, 5 Nov 2020 17:34:01 +0300 Message-Id: In-Reply-To: References: MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Subject: [Tarantool-patches] [PATCH v1 2/2] test: fix flaky election_qsync_stress with KILL List-Id: Tarantool development patches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Kirill Yukhin Cc: tarantool-patches@dev.tarantool.org Found that replication/election_qsync_stress.test.lua test may fail on restating instances. It occures on heavy loaded hosts when its local call to stop instance using SIGTERM fails to stop it. Decided to use SIGKILL in local stop call options to be sure that the instance will be stopped. --- test/replication/election_qsync_stress.result | 2 +- test/replication/election_qsync_stress.test.lua | 2 +- test/replication/suite.ini | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a/test/replication/election_qsync_stress.result b/test/replication/election_qsync_stress.result index b54dc23c4..598e6a9ff 100644 --- a/test/replication/election_qsync_stress.result +++ b/test/replication/election_qsync_stress.result @@ -93,7 +93,7 @@ for i = 1,10 do c:eval('box.cfg{replication_synchro_quorum=4, replication_synchro_timeout=1000}') c.space.test:insert({i}, {is_async=true}) test_run:wait_cond(function() return c.space.test:get{i} ~= nil end) - test_run:cmd('stop server '..old_leader) + test_run:cmd('stop server '..old_leader..' with signal=KILL') nrs[old_leader_nr] = false new_leader_nr = get_leader(nrs) new_leader = 'election_replica'..new_leader_nr diff --git a/test/replication/election_qsync_stress.test.lua b/test/replication/election_qsync_stress.test.lua index 873aabf00..384e36134 100644 --- a/test/replication/election_qsync_stress.test.lua +++ b/test/replication/election_qsync_stress.test.lua @@ -54,7 +54,7 @@ for i = 1,10 do c:eval('box.cfg{replication_synchro_quorum=4, replication_synchro_timeout=1000}') c.space.test:insert({i}, {is_async=true}) test_run:wait_cond(function() return c.space.test:get{i} ~= nil end) - test_run:cmd('stop server '..old_leader) + test_run:cmd('stop server '..old_leader..' with signal=KILL') nrs[old_leader_nr] = false new_leader_nr = get_leader(nrs) new_leader = 'election_replica'..new_leader_nr diff --git a/test/replication/suite.ini b/test/replication/suite.ini index a209f0f3c..8d3a330aa 100644 --- a/test/replication/suite.ini +++ b/test/replication/suite.ini @@ -119,7 +119,7 @@ fragile = { }, "election_qsync_stress.test.lua": { "issues": [ "gh-5395" ], - "checksums": [ "133676d72249c570f7124440150a8790", "83e1b4d48bd095590283247352b37ebb", "358bf14addaee9c8f2dbfe64374bb771", "e217688aa8c995123506662a1b07f2d1", "7cbb6b62027df3d1f31ea94f298bc2e0" ] + "checksums": [ "133676d72249c570f7124440150a8790", "83e1b4d48bd095590283247352b37ebb", "358bf14addaee9c8f2dbfe64374bb771", "e217688aa8c995123506662a1b07f2d1", "7cbb6b62027df3d1f31ea94f298bc2e0", "353dcc43b70c2b34fdea29c7ab10d018", "82cf01d9b40e877c6a93448acb4f721d", "d02417bf4344dccd14601b3bdc7a4798", "18662765b1a8db977817f0d3527dad2f", "c0710e3023f47aee8785c9096c181f51" ] }, "gh-3711-misc-no-restart-on-same-configuration.test.lua": { "issues": [ "gh-5407" ], -- 2.25.1