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

Kirill Yukhin kyukhin at tarantool.org
Fri Dec 4 12:35:44 MSK 2020


Hello,

On 04 Dec 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

I've checked your patch into 1.10 2.5 2.6 and master.

--
Regards, Kirill Yukhin


More information about the Tarantool-patches mailing list