From: "Alexander V. Tikhonov" <avtikhon@tarantool.org> To: Kirill Yukhin <kyukhin@tarantool.org> Cc: tarantool-patches@dev.tarantool.org Subject: [Tarantool-patches] [PATCH v1] test: flaky sql/gh2483-remote-persistency-* test Date: Wed, 11 Nov 2020 08:50:07 +0300 [thread overview] Message-ID: <9e40e6289507355f737205c153344dad786285db.1605073344.git.avtikhon@tarantool.org> (raw) Found issue: No output during 120 seconds. Will abort after 120 seconds without output. List of workers not reporting the status: Test hung! Result content mismatch: - [1] ... test_run:cmd("restart server default") --- Connect to ourself It happened because test-run used to restart 'tarantoolctl restart' routine with default signal SIGTERM. After tarantoolctl updated with the ability to provide needed signal to kill the instance, then SIGKILL was set in the test. Part of #5510 --- Github: https://github.com/tarantool/tarantool/tree/avtikhon/gh-5510-hang-gh2483 Issue: https://github.com/tarantool/tarantool/issues/5510 test/sql/gh2483-remote-persistency-check.result | 2 +- test/sql/gh2483-remote-persistency-check.test.lua | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/test/sql/gh2483-remote-persistency-check.result b/test/sql/gh2483-remote-persistency-check.result index 3ee49b349..185629827 100644 --- a/test/sql/gh2483-remote-persistency-check.result +++ b/test/sql/gh2483-remote-persistency-check.result @@ -32,7 +32,7 @@ box.execute([[SELECT * FROM t]]) rows: - [1] ... -test_run:cmd('restart server default'); +test_run:cmd('restart server default with signal=KILL'); -- Connect to ourself c = require('net.box').connect(os.getenv("LISTEN")) --- diff --git a/test/sql/gh2483-remote-persistency-check.test.lua b/test/sql/gh2483-remote-persistency-check.test.lua index b36b402fb..fca8dad47 100644 --- a/test/sql/gh2483-remote-persistency-check.test.lua +++ b/test/sql/gh2483-remote-persistency-check.test.lua @@ -13,7 +13,7 @@ box.execute([[INSERT INTO t (id) VALUES (1)]]) -- Sanity check box.execute([[SELECT * FROM t]]) -test_run:cmd('restart server default'); +test_run:cmd('restart server default with signal=KILL'); -- Connect to ourself c = require('net.box').connect(os.getenv("LISTEN")) -- 2.25.1
reply other threads:[~2020-11-11 5:50 UTC|newest] Thread overview: [no followups] expand[flat|nested] mbox.gz Atom feed
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=9e40e6289507355f737205c153344dad786285db.1605073344.git.avtikhon@tarantool.org \ --to=avtikhon@tarantool.org \ --cc=kyukhin@tarantool.org \ --cc=tarantool-patches@dev.tarantool.org \ --subject='Re: [Tarantool-patches] [PATCH v1] test: flaky sql/gh2483-remote-persistency-* test' \ /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