Skip to content

Commit

Permalink
ci: static-checks: Move "make check" to the new test matrix
Browse files Browse the repository at this point in the history
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: kata-containers#7974 -- part 0

Signed-off-by: Fabiano Fidêncio <[email protected]>
  • Loading branch information
fidencio committed Sep 16, 2023
1 parent 473ec87 commit bf888b9
Showing 1 changed file with 6 additions and 4 deletions.
10 changes: 6 additions & 4 deletions .github/workflows/static-checks.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,7 @@ jobs:
echo "Check passed"
fi
check-vendor:
build-checks:
runs-on: ubuntu-20.04
strategy:
fail-fast: false
Expand All @@ -49,6 +49,9 @@ jobs:
- log-parser-rs
- runk
- trace-forwarder
command:
- "make vendor"
- "make check"
include:
- component: agent
component-path: src/agent
Expand Down Expand Up @@ -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"

Expand All @@ -105,7 +108,6 @@ jobs:
matrix:
cmd:
- "make static-checks"
- "make check"
- "make test"
- "sudo -E PATH=\"$PATH\" make test"
env:
Expand Down

0 comments on commit bf888b9

Please sign in to comment.