Skip to content

Commit

Permalink
ci: extend tarantool integration testing
Browse files Browse the repository at this point in the history
This patch makes use of reusable workflows from the
Tarantool repository, so now LuaJIT CI tests integration
in all relevant workflows.

Also, this patch disables MacOS testing for Tarantool
integration since those runs were made nightly in the
Tarantool repo.

As noted in the documentation[1], it is impossible for a single
workflow call tree to include more than 20 workflows. To
overcome this limitation, workflows are started in a bunch of
separate files. This makes it impossible to depend on LuaJIT-only
jobs for integration workflow since it is impossible to make
cross-dependencies between workflow files.

The name of a callee workflow cannot be substituted using a
matrix[2], so workflow calls are copied and pasted instead.

Table below shows which workflows are enabled and why.

Workflow name                   |+/-| Reason
-----------------------------------------------------------------
codeql                          | - | Not relevant to LuaJIT.
coverage                        | + | Long tests for profilers.
coverity                        | - | Cron workflow.
debug                           | + | Tarantool debug build.
debug_aarch64                   | + | Tarantool debug build.
debug_asan_clang                | + | Tarantool debug build.
default_gcc_centos_7            | + | Ancient gcc build.
freebsd                         | - | Nightly build.
fuzzing                         | - | Impossible to bump LuaJIT.
integration                     | + | Tarantool ecosystem.
jepsen-cluster-txm              | - | Manual workflow.
jepsen-cluster                  | - | Manual workflow.
jepsen-single-instance-txm      | - | Cron workflow.
jepsen-single-instance          | - | Cron workflow.
lango-stale-reviews             | - | Cron workflow.
lint                            | - | LuaJIT has its own lint.
luajit-integration              | + | Exotic LuaJIT options.
memtx_allocator_based_on_malloc | + | Not relevant to LuaJIT.
osx                             | - | Nightly build.
out_of_source                   | - | Not relevent to LuaJIT.
packaging                       | - | No LuaJIT-relevant variety.
perf_cbench                     | - | Not enabled for PRs.
perf_linkbench_ssd              | - | Not enabled for PRs.
perf_micro                      | - | Not relevant to LuaJIT.
perf_nosqlbench_hash            | - | Not enabled for PRs.
perf_nosqlbench_tree            | - | Not enabled for PRs.
perf_sysbench                   | - | Not enabled for PRs.
perf_tpcc                       | - | Not enabled for PRs.
perf_tpch                       | - | Not enabled for PRs.
perf_ycsb_hash                  | - | Not enabled for PRs.
perf_ycsb_tree                  | - | Not enabled for PRs.
publish-module-api-doc          | - | No Doxygen in LuaJIT.
release                         | + | Tarantool release build.
release_asan_clang              | + | Tarantool release build.
release_clang                   | + | Tarantool release build.
release_lto                     | + | Tarantool release build.
release_lto_clang               | + | Tarantool release build.
reusable_build                  | - | Utility for integration.
source                          | - | Not enabled for PRs.
static_build                    | + | Tarantool static build.
static_build_cmake_linux        | + | Tarantool static build.
static_build_pack_test_deploy   | - | Utility for packaging.
submodule_update                | - | Not enabled for PRs.

[1]: https://docs.github.com/en/actions/using-workflows/reusing-workflows#limitations
[2]: https://github.com/orgs/community/discussions/45342#discussioncomment-4779360
  • Loading branch information
mkokryashkin committed Feb 29, 2024
1 parent e316cbf commit 0291ff9
Show file tree
Hide file tree
Showing 3 changed files with 183 additions and 30 deletions.
62 changes: 62 additions & 0 deletions .github/workflows/exotic-tarantool.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,62 @@
name: "Exotic Tarantool builds testing"

on:
push:
branches-ignore:
- '**-notest'
- 'upstream-**'
tags-ignore:
- '**'

concurrency:
# An update of a developer branch cancels the previously
# scheduled workflow run for this branch. However, the default
# branch, and long-term branch (tarantool/release/2.11,
# tarantool/release/2.10, etc) workflow runs are never canceled.
#
# We use a trick here: define the concurrency group as 'workflow
# run ID' + # 'workflow run attempt' because it is a unique
# combination for any run. So it effectively discards grouping.
#
# XXX: we cannot use `github.sha` as a unique identifier because
# pushing a tag may cancel a run that works on a branch push
# event.
group: ${{ startsWith(github.ref, 'refs/heads/tarantool/')
&& format('{0}-{1}', github.run_id, github.run_attempt)
|| format('{0}-{1}', github.workflow, github.ref) }}
cancel-in-progress: true

jobs:
test-exotic-tarantool:
strategy:
fail-fast: false
matrix:
ARCH: [ARM64, x86_64]
BUILDTYPE: [Debug, Release]
GC64: [ON, OFF]
OS: [Linux, macOS]
include:
- BUILDTYPE: Debug
CMAKEFLAGS: -DCMAKE_BUILD_TYPE=Debug
- BUILDTYPE: Release
CMAKEFLAGS: -DCMAKE_BUILD_TYPE=RelWithDebInfo
exclude:
- ARCH: ARM64
GC64: OFF
- OS: macOS
GC64: OFF
name: >
Tarantool
(${{ matrix.OS }}/${{ matrix.ARCH }})
${{ matrix.BUILDTYPE }}
GC64:${{ matrix.GC64 }}
# XXX: Only LuaJIT-tests are running in this workflow.
uses: tarantool/tarantool/.github/workflows/luajit-integration.yml@master
with:
CMAKE_EXTRA_PARAMS: >
-G Ninja
${{ matrix.CMAKEFLAGS }}
-DLUAJIT_ENABLE_GC64=${{ matrix.GC64 }}
arch: ${{ matrix.ARCH }}
os: ${{ matrix.OS }}
revision: ${{ github.sha }}
35 changes: 35 additions & 0 deletions .github/workflows/tarantool-ecosystem.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,35 @@
name: Tarantool ecosystem integration testing

on:
push:
branches-ignore:
- '**-notest'
- 'upstream-**'
tags-ignore:
- '**'

concurrency:
# An update of a developer branch cancels the previously
# scheduled workflow run for this branch. However, the default
# branch, and long-term branch (tarantool/release/2.11,
# tarantool/release/2.10, etc) workflow runs are never canceled.
#
# We use a trick here: define the concurrency group as 'workflow
# run ID' + # 'workflow run attempt' because it is a unique
# combination for any run. So it effectively discards grouping.
#
# XXX: we cannot use `github.sha` as a unique identifier because
# pushing a tag may cancel a run that works on a branch push
# event.
group: ${{ startsWith(github.ref, 'refs/heads/tarantool/')
&& format('{0}-{1}', github.run_id, github.run_attempt)
|| format('{0}-{1}', github.workflow, github.ref) }}
cancel-in-progress: true

jobs:
test-tarantool-integration:
uses: tarantool/tarantool/.github/workflows/integration.yml@master
with:
submodule: luajit
revision: ${{ github.sha }}

116 changes: 86 additions & 30 deletions .github/workflows/testing.yml
Original file line number Diff line number Diff line change
Expand Up @@ -83,37 +83,93 @@ jobs:
run: cmake --build . --parallel --target LuaJIT-test
working-directory: ${{ env.BUILDDIR }}

test-tarantool-coverage:
needs: test-luajit
uses: tarantool/tarantool/.github/workflows/coverage.yml@master
with:
submodule: luajit
revision: ${{ github.sha }}

test-tarantool:
strategy:
fail-fast: false
matrix:
ARCH: [ARM64, x86_64]
BUILDTYPE: [Debug, Release]
GC64: [ON, OFF]
OS: [Linux, macOS]
include:
- BUILDTYPE: Debug
CMAKEFLAGS: -DCMAKE_BUILD_TYPE=Debug
- BUILDTYPE: Release
CMAKEFLAGS: -DCMAKE_BUILD_TYPE=RelWithDebInfo
exclude:
- ARCH: ARM64
GC64: OFF
- OS: macOS
GC64: OFF
name: >
Tarantool
(${{ matrix.OS }}/${{ matrix.ARCH }})
${{ matrix.BUILDTYPE }}
GC64:${{ matrix.GC64 }}
test-tarantool-debug:
needs: test-luajit
uses: tarantool/tarantool/.github/workflows/debug.yml@master
with:
submodule: luajit
revision: ${{ github.sha }}

test-tarantool-debug_aarch64:
needs: test-luajit
uses: tarantool/tarantool/.github/workflows/debug_aarch64.yml@master
with:
submodule: luajit
revision: ${{ github.sha }}

test-tarantool-debug_asan_clang:
needs: test-luajit
uses: tarantool/tarantool/.github/workflows/debug_asan_clang.yml@master
with:
submodule: luajit
revision: ${{ github.sha }}

test-tarantool-default_gcc_centos_7:
needs: test-luajit
uses: tarantool/tarantool/.github/workflows/default_gcc_centos_7.yml@master
with:
submodule: luajit
revision: ${{ github.sha }}

test-tarantool-out_of_source:
needs: test-luajit
uses: tarantool/tarantool/.github/workflows/out_of_source.yml@master
with:
submodule: luajit
revision: ${{ github.sha }}

test-tarantool-release:
needs: test-luajit
uses: tarantool/tarantool/.github/workflows/release.yml@master
with:
submodule: luajit
revision: ${{ github.sha }}

test-tarantool-release_asan_clang:
needs: test-luajit
uses: tarantool/tarantool/.github/workflows/release_asan_clang.yml@master
with:
submodule: luajit
revision: ${{ github.sha }}

test-tarantool-release_clang:
needs: test-luajit
uses: tarantool/tarantool/.github/workflows/release_clang.yml@master
with:
submodule: luajit
revision: ${{ github.sha }}

test-tarantool-release_lto:
needs: test-luajit
uses: tarantool/tarantool/.github/workflows/release_lto.yml@master
with:
submodule: luajit
revision: ${{ github.sha }}

test-tarantool-release_lto_clang:
needs: test-luajit
uses: tarantool/tarantool/.github/workflows/release_lto_clang.yml@master
with:
submodule: luajit
revision: ${{ github.sha }}

test-tarantool-static_build:
needs: test-luajit
uses: tarantool/tarantool/.github/workflows/static_build.yml@master
with:
submodule: luajit
revision: ${{ github.sha }}

test-tarantool-static_build_cmake_linux:
needs: test-luajit
uses: tarantool/tarantool/.github/workflows/luajit-integration.yml@master
uses: tarantool/tarantool/.github/workflows/static_build_cmake_linux.yml@master
with:
CMAKE_EXTRA_PARAMS: >
-G Ninja
${{ matrix.CMAKEFLAGS }}
-DLUAJIT_ENABLE_GC64=${{ matrix.GC64 }}
arch: ${{ matrix.ARCH }}
os: ${{ matrix.OS }}
submodule: luajit
revision: ${{ github.sha }}

0 comments on commit 0291ff9

Please sign in to comment.