[Tarantool-patches] [PATCH luajit 0/2] Fix tests for LUAJIT_ENABLE_CHECKHOOK

Igor Munkin imun at tarantool.org
Fri Dec 2 13:05:01 MSK 2022


I've extended the series with additional patch, fixing flaky subtest in
test/tarantool-tests/misclib-getmetrics-lapi.test.lua. I failed to find
the root cause why it became flaky as a result of enabling the new
option LUAJIT_ENABLE_CHECKHOOK, but the test was implemented in a
fragile way, so its flakiness was a matter of time. Fixed now, updated
both LuaJIT branch and Tarantool PR.

On 01.12.22, Igor Munkin wrote:
> After enabling LUAJIT_ENABLE_CHECKHOOK options two tests started to fail
> due to very tight JIT engine setup. This patchset is aimed to relax JIT
> engine setup and make the tests agnostic to the introduced option.
> 
> Issue: https://github.com/tarantool/tarantool/issues/7762
> Branch: https://github.com/tarantool/luajit/tree/imun/gh-7762-fix-tests-for-checkhook
> PR: https://github.com/tarantool/tarantool/pull/7787
> CI: https://github.com/tarantool/luajit/commit/ba9d9c8
> 
> Igor Munkin (2):
>   test: relax JIT setup in lj-430-maxirconst test
>   test: relax JIT setup in misc.getmetrics test
> 
>  .../lj-430-maxirconst.test.lua                | 23 +++++++++++--------
>  .../misclib-getmetrics-capi.test.lua          |  7 ++++--
>  .../misclib-getmetrics-lapi.test.lua          |  7 ++++--
>  test/tarantool-tests/utils.lua                |  9 ++++++++
>  4 files changed, 33 insertions(+), 13 deletions(-)
> 
> -- 
> 2.34.0
> 

-- 
Best regards,
IM


More information about the Tarantool-patches mailing list