[Tarantool-patches] [PATCH 3/3] ci: enable LuaJIT tests in CI

Igor Munkin imun at tarantool.org
Sat Feb 20 13:18:38 MSK 2021


Timur,

On 20.02.21, Timur Safin wrote:
> : From: Tarantool-patches <tarantool-patches-bounces at dev.tarantool.org> On
> : Subject: Re: [Tarantool-patches] [PATCH 3/3] ci: enable LuaJIT tests in CI
> : 
> : Sergey,
> : 
> : Thanks for your review!
> : > Nit: There is one general comment for the whole patch:
> : > In my opinion, it is better to run `make LuaJIT-tests` **before**
> : > Tarantool test suite. If LuaJIT is broken failed Tarantool tests
> : > information is useless (Is it LuaJIT? Always has been.).
> : > Feel free to ignore.
> : 
> : I assume the following: there is no flaky tests in LuaJIT (at least now)
> : but there are many in Tarantool. If we place LuaJIT-related tests before
> : Tarantool tests, then we need to rerun the former if the latter fail.
> : But when Tarantool tests fully succeed (even with no test-run internal
> : issues), LuaJIT tests are unlikely to fail. This makes sense when LuaJIT
> : tests will grow in a nearest future.
> : 
> : Ignoring for now but let's return to the question if it is necessary.
> : 
> 
> Here I'd rather agree with Sergey, that LuaJIT tests are prerequisite for
> all lua testing in Tarantool.

Then OK. I reordered the commands, squashed, force-pushed to the branch.

> 
> Timur
> 

-- 
Best regards,
IM


More information about the Tarantool-patches mailing list