From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from [87.239.111.99] (localhost [127.0.0.1]) by dev.tarantool.org (Postfix) with ESMTP id 8C954D47B80; Fri, 15 Mar 2024 13:56:56 +0300 (MSK) DKIM-Filter: OpenDKIM Filter v2.11.0 dev.tarantool.org 8C954D47B80 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=tarantool.org; s=dev; t=1710500216; bh=nPft4eHdD90XlYia0wkUEiukqBeErwhIGJKZjIB3ru8=; h=To:Date:Subject:List-Id:List-Unsubscribe:List-Archive:List-Post: List-Help:List-Subscribe:From:Reply-To:From; b=S79eq91pjrhfchmi/uSrdMFF0tBdk2Zg+Yx4ekeLUAcAbClL3CbvuehKbjTHncaf4 xx2hx+XGIuHil6XUuLKMxPKMuwddVXN5lqFp9zX644aW3UkWfVa1bAbMjyRHsXBKQ/ MwbipSDD3MIIS+kEwD+4AGarulBRL2uGuyrU/zIU= Received: from mail-lf1-f52.google.com (mail-lf1-f52.google.com [209.85.167.52]) (using TLSv1.3 with cipher TLS_AES_128_GCM_SHA256 (128/128 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits) server-digest SHA256) (No client certificate requested) by dev.tarantool.org (Postfix) with ESMTPS id 8A796D47B80 for ; Fri, 15 Mar 2024 13:56:55 +0300 (MSK) DKIM-Filter: OpenDKIM Filter v2.11.0 dev.tarantool.org 8A796D47B80 Received: by mail-lf1-f52.google.com with SMTP id 2adb3069b0e04-513d4559fb4so1849980e87.3 for ; Fri, 15 Mar 2024 03:56:55 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1710500214; x=1711105014; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=sYZx6Qg+5aJ7fHw9iwPBvwzekLLnRMK8e/TKENl+jQE=; b=BnjbLbm/ZWjeUX9jdO7wtKzxyytsYGiNbmwyVhh2JGBMQ03Ik94F+y3BA1Sc/jlPaf ofKCr4fUQac2sPwuVd5kzUV95a3VPbgBJdcxdkod//DPk5I6yz9hEioMuR516XM4ZxOx w7HIPkElxXjnIPLkZ3846QOlxcRnyRJtPI+w6/XTKV+PvNDSgbBExtBWZ8BYuqZVkC1U prL+5BImoA+Oe5ZeaRLDugjRqa1wIX2PeM6/aMDK2Mmjvu37pXSEqug0Yn3eowX5j7/j rHmX+ixZY2eaVbIqIV5fEi/dFNQtrV29LKY7vH/hLxBUE/xZbM1wxVc8JFG2KH8ZJiZd G5gA== X-Gm-Message-State: AOJu0YxMFKS3s7y1Z3Z0vZnWxCE6j3onlfsCSwfzZG80LQDQwfyThasE laoXNFK48anEVtaWKD2LbGjokEvB2NGLHGcyv6cnP4zHhEGQWv7Z0oLKhAHCs0o3xg== X-Google-Smtp-Source: AGHT+IEq0oF4IprjmlcLpW+MS9p5+2CgJWTV8JfO3Cyfy7ZlTJ9jfpl890o8z33FWT/+rsuCuRkleQ== X-Received: by 2002:a19:e057:0:b0:513:d50f:23db with SMTP id g23-20020a19e057000000b00513d50f23dbmr1991199lfj.31.1710500214119; Fri, 15 Mar 2024 03:56:54 -0700 (PDT) Received: from localhost.localdomain ([2a00:1fa0:64a:26f4:d894:e5ee:9ee5:bc2a]) by smtp.gmail.com with ESMTPSA id b2-20020a056512024200b00513216fa562sm605263lfo.202.2024.03.15.03.56.52 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 15 Mar 2024 03:56:53 -0700 (PDT) X-Google-Original-From: Maxim Kokryashkin To: tarantool-patches@dev.tarantool.org, skaplun@tarantool.org, sergeyb@tarantool.org Date: Fri, 15 Mar 2024 13:56:50 +0300 Message-ID: <20240315105650.31960-1-m.kokryashkin@tarantool.org> X-Mailer: git-send-email 2.44.0 MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Subject: [Tarantool-patches] [PATCH luajit/ Backport 2.11] ci: extend tarantool integration testing X-BeenThere: tarantool-patches@dev.tarantool.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: Tarantool development patches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , From: Maxim Kokryashkin via Tarantool-patches Reply-To: Maxim Kokryashkin Errors-To: tarantool-patches-bounces@dev.tarantool.org Sender: "Tarantool-patches" 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 workflows such as `tarantool-ecosystem.yml` or `tarantool-exotic.yml` 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 ----------------------------------------------------------------- almalinux_8_aarch64 | - | No LuaJIT-relevant variety. almalinux_8 | - | No LuaJIT-relevant variety. almalinux_9_aarch64 | - | No LuaJIT-relevant variety. almalinux_9 | - | No LuaJIT-relevant variety. alpine_3_16_aarch64 | - | No LuaJIT-relevant variety. alpine_3_16 | - | No LuaJIT-relevant variety. centos_7_aarch64 | - | No LuaJIT-relevant variety. centos_7 | - | No LuaJIT-relevant variety. centos_8_aarch64 | - | No LuaJIT-relevant variety. centos_8 | - | No LuaJIT-relevant variety. codeql | - | Not relevant to LuaJIT. coverage | + | Long tests for profilers. coverity | - | Cron workflow. debian_10_aarch64 | - | No LuaJIT-relevant variety. debian_10 | - | No LuaJIT-relevant variety. debian_11_aarch64 | - | No LuaJIT-relevant variety. debian_11 | - | No LuaJIT-relevant variety. debian_12_aarch64 | - | No LuaJIT-relevant variety. debian_12 | - | No LuaJIT-relevant variety. debian_9 | - | No LuaJIT-relevant variety. debug | + | Tarantool debug build. debug_aarch64 | + | Tarantool debug build. debug_asan_clang | + | Tarantool debug build. default_gcc_centos_7 | + | Old gcc version (7). fedora_34_aarch64 | - | No LuaJIT-relevant variety. fedora_34 | - | No LuaJIT-relevant variety. fedora_35_aarch64 | - | No LuaJIT-relevant variety. fedora_35 | - | No LuaJIT-relevant variety. fedora_36_aarch64 | - | No LuaJIT-relevant variety. fedora_36 | - | No LuaJIT-relevant variety. fedora_37_aarch64 | - | No LuaJIT-relevant variety. fedora_37 | - | No LuaJIT-relevant variety. fedora_38_aarch64 | - | No LuaJIT-relevant variety. fedora_38 | - | No LuaJIT-relevant variety. fedora_39_aarch64 | - | No LuaJIT-relevant variety. fedora_39 | - | No LuaJIT-relevant variety. 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 | + | Run tests under Tarantool. memtx_allocator_based_on_malloc | - | Not relevant to LuaJIT. osx | - | Nightly build. out_of_source | + | Out of source build. 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. redos_7_3.yaml | - | No LuaJIT-relevant variety. 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 | - | Just an OOS static build. static_build_pack_test_deploy | - | Utility for packaging. submodule_update | - | Not enabled for PRs. ubuntu_16_04 | - | No LuaJIT-relevant variety. ubuntu_18_04 | - | No LuaJIT-relevant variety. ubuntu_20_04_aarch64 | - | No LuaJIT-relevant variety. ubuntu_20_04 | - | No LuaJIT-relevant variety. ubuntu_22_04_aarch64 | - | No LuaJIT-relevant variety. ubuntu_22_04 | - | No LuaJIT-relevant variety. [1]: https://docs.github.com/en/actions/using-workflows/reusing-workflows#limitations [2]: https://github.com/orgs/community/discussions/45342#discussioncomment-4779360 (cherry picked from commit 0dfa2cec5febc7a1796388b6e96a7b58b9006d51) --- Branch: https://github.com/tarantool/luajit/tree/mkokryashkin/integration-testing-2.11 .../integration-tarantool-ecosystem.yml | 41 ++++++ .github/workflows/integration-tarantool.yml | 118 ++++++++++++++++++ .github/workflows/testing.yml | 9 ++ 3 files changed, 168 insertions(+) create mode 100644 .github/workflows/integration-tarantool-ecosystem.yml create mode 100644 .github/workflows/integration-tarantool.yml diff --git a/.github/workflows/integration-tarantool-ecosystem.yml b/.github/workflows/integration-tarantool-ecosystem.yml new file mode 100644 index 00000000..5ad71260 --- /dev/null +++ b/.github/workflows/integration-tarantool-ecosystem.yml @@ -0,0 +1,41 @@ +# XXX: A single call tree for reusable workflows can't exceed the +# total of 20 workflows, as stated in the documentation [1]. +# This workflow file was created to avoid this limitation. +# +# [1]: https://docs.github.com/en/actions/using-workflows/reusing-workflows#limitations +name: 'Integration / Tarantool ecosystem' + +on: + push: + branches-ignore: + - '**-notest' + - '**-nointegration' + - '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@release/2.11 + secrets: inherit + with: + submodule: luajit + revision: ${{ github.sha }} diff --git a/.github/workflows/integration-tarantool.yml b/.github/workflows/integration-tarantool.yml new file mode 100644 index 00000000..5ab84b93 --- /dev/null +++ b/.github/workflows/integration-tarantool.yml @@ -0,0 +1,118 @@ +# XXX: A single call tree for reusable workflows can't exceed the +# total of 20 workflows, as stated in the documentation [1]. +# This workflow file was created to avoid this limitation. +# +# [1]: https://docs.github.com/en/actions/using-workflows/reusing-workflows#limitations +name: 'Integration / Tarantool' + +on: + push: + branches-ignore: + - '**-notest' + - '**-nointegration' + - '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-coverage: + name: coverage + uses: tarantool/tarantool/.github/workflows/coverage.yml@release/2.11 + with: + submodule: luajit + revision: ${{ github.sha }} + + test-tarantool-debug: + name: debug + uses: tarantool/tarantool/.github/workflows/debug.yml@release/2.11 + with: + submodule: luajit + revision: ${{ github.sha }} + + test-tarantool-debug_aarch64: + name: debug aarch64 + uses: tarantool/tarantool/.github/workflows/debug_aarch64.yml@release/2.11 + with: + submodule: luajit + revision: ${{ github.sha }} + + test-tarantool-debug_asan_clang: + name: debug ASAN clang + uses: tarantool/tarantool/.github/workflows/debug_asan_clang.yml@release/2.11 + with: + submodule: luajit + revision: ${{ github.sha }} + + test-tarantool-default_gcc_centos_7: + name: gcc centos 7 + uses: tarantool/tarantool/.github/workflows/default_gcc_centos_7.yml@release/2.11 + with: + submodule: luajit + revision: ${{ github.sha }} + + test-tarantool-out_of_source: + name: out of source + uses: tarantool/tarantool/.github/workflows/out_of_source.yml@release/2.11 + with: + submodule: luajit + revision: ${{ github.sha }} + + test-tarantool-release: + name: release + uses: tarantool/tarantool/.github/workflows/release.yml@release/2.11 + with: + submodule: luajit + revision: ${{ github.sha }} + + test-tarantool-release_asan_clang: + name: release ASAN clang + uses: tarantool/tarantool/.github/workflows/release_asan_clang.yml@release/2.11 + with: + submodule: luajit + revision: ${{ github.sha }} + + test-tarantool-release_clang: + name: release clang + uses: tarantool/tarantool/.github/workflows/release_clang.yml@release/2.11 + with: + submodule: luajit + revision: ${{ github.sha }} + + test-tarantool-release_lto: + name: release lto + uses: tarantool/tarantool/.github/workflows/release_lto.yml@release/2.11 + with: + submodule: luajit + revision: ${{ github.sha }} + + test-tarantool-release_lto_clang: + name: release lto clang + uses: tarantool/tarantool/.github/workflows/release_lto_clang.yml@release/2.11 + with: + submodule: luajit + revision: ${{ github.sha }} + + test-tarantool-static_build: + name: static build + uses: tarantool/tarantool/.github/workflows/static_build.yml@release/2.11 + with: + submodule: luajit + revision: ${{ github.sha }} diff --git a/.github/workflows/testing.yml b/.github/workflows/testing.yml index 031a153f..c6fa1105 100644 --- a/.github/workflows/testing.yml +++ b/.github/workflows/testing.yml @@ -1,3 +1,11 @@ +# XXX: A single call tree for reusable workflows can't exceed the +# total of 20 workflows, as stated in the documentation [1]. +# Some other jobs are started in separate workflow files to +# overcome the limitation. The jobs are Tarantool ecosystem +# integration testing (tarantool-ecosystem.yml) and exotic +# Tarantool builds (tarantool-exotic.yml). +# +# [1]: https://docs.github.com/en/actions/using-workflows/reusing-workflows#limitations name: Testing on: @@ -108,6 +116,7 @@ jobs: ${{ matrix.BUILDTYPE }} GC64:${{ matrix.GC64 }} needs: test-luajit + # XXX: Only LuaJIT-tests are running in this workflow. uses: tarantool/tarantool/.github/workflows/luajit-integration.yml@master with: CMAKE_EXTRA_PARAMS: > -- 2.44.0