Tarantool development patches archive
 help / color / mirror / Atom feed
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