Draft
Conversation
Totktonada
reviewed
Jan 27, 2025
8bd3d02 to
5e64868
Compare
5e64868 to
48feabf
Compare
fdbd8ef to
f6640b1
Compare
ligurio
added a commit
to ligurio/nanodata
that referenced
this pull request
Mar 28, 2025
The commit 8ad443d ("cmake: introduce a module to build fuzzing tests") has added tests for LuaJIT. These tests mostly written in C and tests LuaJIT via Lua C API. The patch integrates tests that written in Lua and tests LuaJIT via Lua API, see [1]. Depends on: 1. ligurio/lunapark#121 2. tarantool#11056 NO_CHANGELOG=testing NO_DOC=testing
ligurio
added a commit
to ligurio/nanodata
that referenced
this pull request
Apr 9, 2025
The commit 8ad443d ("cmake: introduce a module to build fuzzing tests") has added tests for LuaJIT. These tests mostly written in C and tests LuaJIT via Lua C API. The patch integrates tests that written in Lua and tests LuaJIT via Lua API, see [1]. Depends on: 1. ligurio/lunapark#121 2. tarantool#11056 NO_CHANGELOG=testing NO_DOC=testing
ligurio
added a commit
to ligurio/nanodata
that referenced
this pull request
May 14, 2025
The commit 8ad443d ("cmake: introduce a module to build fuzzing tests") has added tests for LuaJIT. These tests mostly written in C and tests LuaJIT via Lua C API. The patch integrates tests that written in Lua and tests LuaJIT via Lua API, see [1]. Depends on: 1. ligurio/lunapark#121 2. tarantool#11056 NO_CHANGELOG=testing NO_DOC=testing
f6640b1 to
9e9ff04
Compare
The commit 52b82bd ("build: configure parallel jobs") calculates optimal parallel level that depends on CPU number and sets that number as CMAKE_BUILD_PARALLEL_LEVEL. The patch removes calculating NPROC in .test.mk, because after aforementioned commit it is not needed. Follows up commit 52b82bd ("build: configure parallel jobs"). NO_CHANGELOG=ci NO_DOC=ci NO_TEST=ci
faa77da to
23d7fd0
Compare
Running tests with enabled ASAN and LSAN requires defining a number of environment variables (like a path to a file with suppressions or custom sanitizers flags). These environment variables are defined in .test.mk and therefore it is complicated to use these variables with CTest. The patch moves environment variables to `cmake/profile.cmake` and sets them in targets with test-run.py and tests for CTest. NO_CHANGELOG=testing NO_DOC=testing
There is a tool named `direnv` [1], it is an extension for shell, that can load and unload environment variables depending on the current directory. Running regression tests with enabled ASAN and LSAN requires setting environment variables. CTest set these variables in each test, but for running `tarantool` it is reqired setting these variables manually. `direnv` and autogenerated `.envrc` solves this problem - CMake automatically generates a file with actual settings, `direnv` loads this file as soon as a current directory changed to a PROJECT_SOURCE_DIR. Note, the command `direnv allow` must be executed for loading updated `.envrc`. 1. https://direnv.net/ NO_CHANGELOG=testing NO_DOC=testing NO_TEST=testing
23d7fd0 to
194ae4a
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Depends on #11004
NO_CHANGELOG=build
NO_DOC=build
NO_TEST=build