From bf888b9a5eb954561d91c190e6fd352f0eb18914 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Fabiano=20Fid=C3=AAncio?= Date: Fri, 15 Sep 2023 15:21:35 +0200 Subject: [PATCH] ci: static-checks: Move "make check" to the new test matrix MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit We're moving it out of the previous "static-checks" confusing matrix, and adding it to the matrix that was currently being used for the `make vendor` checks. This will allow us to have one job per component, and with that we can easily run those in parallel and on the zero cost runners. Fixes: #7974 -- part 0 Signed-off-by: Fabiano FidĂȘncio --- .github/workflows/static-checks.yaml | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/.github/workflows/static-checks.yaml b/.github/workflows/static-checks.yaml index 3938691741..b2de664acd 100644 --- a/.github/workflows/static-checks.yaml +++ b/.github/workflows/static-checks.yaml @@ -34,7 +34,7 @@ jobs: echo "Check passed" fi - check-vendor: + build-checks: runs-on: ubuntu-20.04 strategy: fail-fast: false @@ -49,6 +49,9 @@ jobs: - log-parser-rs - runk - trace-forwarder + command: + - "make vendor" + - "make check" include: - component: agent component-path: src/agent @@ -88,10 +91,10 @@ jobs: run: | ./tests/install_rust.sh echo "${HOME}/.cargo/bin" >> $GITHUB_PATH - - name: Check ${{ matrix.component }} vendored code + - name: Running `${{ matrix.command }}` for ${{ matrix.component }} run: | cd ${{ matrix.component-path }} - make vendor + ${{ matrix.command }} env: RUST_BACKTRACE: "1" @@ -105,7 +108,6 @@ jobs: matrix: cmd: - "make static-checks" - - "make check" - "make test" - "sudo -E PATH=\"$PATH\" make test" env: