Re: [Tarantool-patches] [PATCH v2 0/2] test: flaky hang vinyl/ddl.test.lua test

[Aleksandr Lyapunov <alyapunov@tarantool.org>]

Hi! thanks for the patch!
LGTM, I understand now.
Please see my minor comments.

On 04.12.2020 11:05, Alexander V. Tikhonov wrote:
> Patchset consists of 2 commits:
>
> 1. Revert "Fix flaky test engine/ddl"
>      
>     Found that the previous fix of the engine/ddl.test.lua test committed
>     with:
>      
>        5f96ee5907672bd32dd7fe8dad144ac3d328187f ('Fix flaky test engine/ddl')
>      
>     did not fix the issue #4353 in real and it was reverted.
>   
> 2. test: flaky hang vinyl/ddl.test.lua test
>      
>     Found hanging test vinyl/ddl.test.lua on:
>      
>       [159]  inspector:wait_cond(function() return box.space.test.index.pk:count() == box.space.test.index.tk:count() end)
>       [159]  ---
>       [159]  - true
>       [159]  ...
>       [159] -box.snapshot()
>       [159] ----
>       [159] -- ok
>       [159] -...
>      
>     The real issue happend before it when test failed on:
>      
>       [091] --- engine/ddl.result   Thu May 14 16:12:09 2020
>       [091] +++ engine/ddl.reject   Fri May 15 04:15:07 2020
>       [091] @@ -2558,7 +2558,7 @@
>       [091]  ...
>       [091]  inspector:wait_cond(function() return box.space.test.index.pk:count() == box.space.test.index.sk:count() end)
>       [091]  ---
>       [091] -- true
>       [091] +- false
>       [091]  ...
>      
>     Our tests have structure when different standalone subtests exists
>     in the test files. To be able to check all of them this hang must
>     be neutralized to give the next standalone subtest ability to pass.
>     To avoid of this hang decided to disable box.snapshot check if the
>     previous check of the current subtest failed.
>      
>     Needed for #4353
>
> Alexander V. Tikhonov (2):
>    Revert "Fix flaky test engine/ddl"
>    test: flaky hang vinyl/ddl.test.lua test
> ---
>
> Github: https://github.com/tarantool/tarantool/tree/avtikhon/gh-4353-hang-ddl
> Issue: https://github.com/tarantool/tarantool/issues/4353
>
>   test/engine/ddl.result   | 91 ++++++++++++++++++++++++++--------------
>   test/engine/ddl.test.lua | 76 ++++++++++++++++++++++++++-------
>   test/engine/suite.ini    |  2 +-
>   3 files changed, 122 insertions(+), 47 deletions(-)
>