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