Refactor CI to return better results faster #278
This file contains 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
name: MLSPP CI | ||
on: | ||
push: | ||
branches: | ||
- main | ||
pull_request: | ||
branches: | ||
- main | ||
env: | ||
CMAKE_BUILD_PARALLEL_LEVEL: 3 | ||
CTEST_OUTPUT_ON_FAILURE: 1 | ||
VCPKG_BINARY_SOURCES: files,${{ github.workspace }}/vcpkg_cache,readwrite | ||
CMAKE_TOOLCHAIN_FILE: ${{ github.workspace }}/vcpkg/scripts/buildsystems/vcpkg.cmake | ||
jobs: | ||
formatting-check: | ||
name: Formatting Check | ||
runs-on: ubuntu-latest | ||
steps: | ||
- uses: actions/checkout@v4 | ||
- name: Run clang-format style check for C/C++ programs | ||
uses: jidicula/[email protected] | ||
with: | ||
clang-format-version: 16 | ||
include-regex: '^\./(src|include|test|cmd)/.*\.(cpp|h)$' | ||
fallback-style: 'Mozilla' | ||
build-and-unit-test: | ||
needs: formatting-check | ||
name: Build and test | ||
runs-on: ${{ matrix.os }} | ||
strategy: | ||
matrix: | ||
os: [windows-latest, ubuntu-latest, macos-latest] | ||
crypto: [openssl_1.1, openssl_3, boringssl] | ||
env: | ||
BUILD_DIR: "${RUNNER_TEMP}/build_${{ matrix.crypto }}" | ||
CRYPTO_DIR: "./alternatives/${{ matrix.crypto }}" | ||
steps: | ||
- uses: actions/checkout@v4 | ||
with: | ||
submodules: recursive | ||
fetch-depth: 0 | ||
- uses: ./.github/actions/prepare-build | ||
with: | ||
os: ${{ matrix.os }} | ||
crypto-dir: ${{ env.CRYPTO_DIR }} | ||
cache-dir: ${{ github.workspace }}/vcpkg_cache | ||
- name: Build | ||
run: | | ||
# XXX(RLB): If we do not have SANITIZERS=ON here, the Windows CI builds | ||
# hang in the middle of unit testing. | ||
cmake -B "${{ env.BUILD_DIR }}" -DVCPKG_MANIFEST_DIR="${{ env.CRYPTO_DIR }}" -DTESTING=ON -DSANITIZERS=ON | ||
cmake --build "${{ env.BUILD_DIR }}" | ||
- name: Unit Test (non-Windows) | ||
if: matrix.os != "windows-latest" | ||
Check failure on line 64 in .github/workflows/main_ci.yml GitHub Actions / MLSPP CIInvalid workflow file
|
||
run: | | ||
cmake --build "${{ env.BUILD_DIR }}" --target test | ||
- name: Unit Test (Windows) | ||
if: matrix.os == "windows-latest" | ||
run: | | ||
cmake --build "${{ env.BUILD_DIR }}" --target RUN_TESTS | ||
interop-test: | ||
if: github.event.pull_request.draft == false | ||
needs: build-and-unit-test | ||
name: Interop test | ||
runs-on: ubuntu-latest | ||
env: | ||
BUILD_DIR: "${RUNNER_TEMP}/build_openssl_1.1" | ||
CRYPTO_DIR: "./alternatives/openssl_1.1" | ||
steps: | ||
- uses: actions/checkout@v4 | ||
with: | ||
submodules: recursive | ||
fetch-depth: 0 | ||
- uses: ./.github/actions/prepare-build | ||
with: | ||
os: ubuntu-latest | ||
crypto-dir: ${{ env.CRYPTO_DIR }} | ||
cache-dir: ${{ github.workspace }}/vcpkg_cache | ||
- name: Build | ||
run: | | ||
cmake -B "${{ env.BUILD_DIR }}" -DVCPKG_MANIFEST_DIR="${{ env.CRYPTO_DIR }}" | ||
cmake --build "${{ env.BUILD_DIR }}" | ||
- name: Build (Interop Harness) | ||
run: | | ||
cd cmd/interop | ||
cmake -B build | ||
cmake --build build | ||
- name: Test self-interop | ||
run: | | ||
make -C cmd/interop self-test | ||
- name: Test interop on test vectors | ||
run: | | ||
make -C cmd/interop interop-test | ||
- name: Test gRPC live interop with self | ||
run: | | ||
cd cmd/interop | ||
./grpc-self-test.sh | ||
clang-tidy: | ||
if: github.event.pull_request.draft == false | ||
needs: build-and-unit-test | ||
name: Build with clang-tidy | ||
runs-on: ubuntu-latest | ||
strategy: | ||
matrix: | ||
crypto: [openssl_1.1, openssl_3, boringssl] | ||
env: | ||
BUILD_DIR: "${RUNNER_TEMP}/build_${{ matrix.crypto }}" | ||
CRYPTO_DIR: "./alternatives/${{ matrix.crypto }}" | ||
steps: | ||
- uses: actions/checkout@v4 | ||
with: | ||
submodules: recursive | ||
fetch-depth: 0 | ||
- uses: ./.github/actions/prepare-build | ||
with: | ||
os: ubuntu-latest | ||
crypto-dir: ${{ env.CRYPTO_DIR }} | ||
cache-dir: ${{ github.workspace }}/vcpkg_cache | ||
- name: Build with clang-tidy | ||
run: | | ||
cmake -B "${{ env.BUILD_DIR }}" -DVCPKG_MANIFEST_DIR="${{ env.CRYPTO_DIR }}" \ | ||
-DTESTING=ON -DCLANG_TIDY=ON -DSANITIZERS=ON | ||
cmake --build "${{ env.BUILD_DIR }}" |