

<HTML><BODY><div>LGTM.<br> <blockquote style="border-left:1px solid #0857A6; margin:10px; padding:0 0 0 10px;">Четверг, 30 апреля 2020, 13:29 +03:00 от Sergey Bronnikov <sergeyb@tarantool.org>:<br> <div id=""><div class="js-helper js-readmsg-msg"><style type="text/css"></style><div><div id="style_15882425752031646492_BODY">Hi, Sasha<br><br>in general LGTM and see comment below.<br><br>On 12:49 Thu 30 Apr , Alexander V. Tikhonov wrote:<br>> Fragiled flaky tests from parallel runs to avoid<br>> of flaky fails in regular testing:<br><br>"fragiled flaky test" is a something like "butter oil"?<br>I propose make commit message more clear. For example:<br>"Mark tests as fragile in a test's configs"<br><br>> snapshot.test.py ; gh-4514<br>> init_storage.test.py ; gh-4949<br>> stat.test.lua ; gh-4951<br>> xlog/checkpoint_daemon.test.lua ; gh-4952<br>><br>> Part of #4953<br>> ---<br>><br>> Github: <a href="https://github.com/tarantool/tarantool/tree/avtikhon/fragile-flaky-tests" target="_blank">https://github.com/tarantool/tarantool/tree/avtikhon/fragile-flaky-tests</a><br>> Issue: <a href="https://github.com/tarantool/tarantool/issues/4953" target="_blank">https://github.com/tarantool/tarantool/issues/4953</a><br>><br>> test/box-py/suite.ini | 1 +<br>> test/replication-py/suite.ini | 1 +<br>> test/vinyl/suite.ini | 1 +<br>> test/xlog/suite.ini | 1 +<br>> 4 files changed, 4 insertions(+)<br>><br>> diff --git a/test/box-py/suite.ini b/test/box-py/suite.ini<br>> index 18737e48f..d1ff0fbfd 100644<br>> --- a/test/box-py/suite.ini<br>> +++ b/test/box-py/suite.ini<br>> @@ -5,3 +5,4 @@ script = box.lua<br>> lua_libs = lua/fiber.lua lua/fifo.lua<br>> use_unix_sockets = True<br>> is_parallel = True<br>> +fragile = snapshot.test.py ; gh-4514<br>> diff --git a/test/replication-py/suite.ini b/test/replication-py/suite.ini<br>> index df2dd8ef0..2658f4816 100644<br>> --- a/test/replication-py/suite.ini<br>> +++ b/test/replication-py/suite.ini<br>> @@ -3,3 +3,4 @@ core = tarantool<br>> script = master.lua<br>> description = tarantool/box, replication<br>> is_parallel = True<br>> +fragile = init_storage.test.py ; gh-4949<br>> diff --git a/test/vinyl/suite.ini b/test/vinyl/suite.ini<br>> index c8bc270f3..44c53dc68 100644<br>> --- a/test/vinyl/suite.ini<br>> +++ b/test/vinyl/suite.ini<br>> @@ -15,3 +15,4 @@ pretest_clean = True<br>> fragile = errinj.test.lua ; gh-4346<br>> select_consistency.test.lua ; gh-4385<br>> throttle.test.lua ; gh-4168<br>> + stat.test.lua ; gh-4951<br>> diff --git a/test/xlog/suite.ini b/test/xlog/suite.ini<br>> index 635ec53c1..8e4c4d374 100644<br>> --- a/test/xlog/suite.ini<br>> +++ b/test/xlog/suite.ini<br>> @@ -10,3 +10,4 @@ use_unix_sockets_iproto = True<br>> long_run = snap_io_rate.test.lua<br>> is_parallel = True<br>> pretest_clean = True<br>> +fragile = xlog/checkpoint_daemon.test.lua ; gh-4952<br>> --<br>> 2.17.1<br>><br><br>--<br>sergeyb@</div></div></div></div></blockquote> <div> </div><div data-signature-widget="container"><div data-signature-widget="content"><div>--<br>Oleg Piskunov</div></div></div><div> </div></div></BODY></HTML>