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-run: restart server on each failed test
Date: Mon,  5 Oct 2020 13:02:43 +0300	[thread overview]
Message-ID: <0bf35c792c1535ed3f55873b8b20345390cf6580.1601892098.git.avtikhon@tarantool.org> (raw)

Found that non restartable workers, used to run groups of tasks, had
only single tarantool server process started in the very start of it
at the worker's initalization. If any test in the worker's queue was
aborted then all the rest tests in queue couldn't be started. To fix
it the server restart commands were added just after the each task
failed. This change could decrease the performance, but it did not,
because in parallel runs these servers restarts don't use resources
and in absolute testing time was not increased. It happened because
there were other bottle necks in resources that made this change
invisible for the performance. Checking the change w/o parallel runs
found that even in this mode the running time wasn't increased too.

https://github.com/tarantool/test-run/pull/225

Closes tarantool/test-run#192
Closes tarantool/test-run#137
Closes tarantool/test-run#215
---

Github: https://github.com/tarantool/tarantool/tree/avtikhon/flaky-checksums

 test-run | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/test-run b/test-run
index 5555485f9..10e2485f7 160000
--- a/test-run
+++ b/test-run
@@ -1 +1 @@
-Subproject commit 5555485f9cb506639fd2be9de2c722e16025a117
+Subproject commit 10e2485f7f0947b2f87aa6a70c843f2f4cd6bfe2
-- 
2.25.1

                 reply	other threads:[~2020-10-05 10:02 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=0bf35c792c1535ed3f55873b8b20345390cf6580.1601892098.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-run: restart server on each failed 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