Skip to content

Commit

Permalink
gh: update cc_kbc CI checks
Browse files Browse the repository at this point in the history
KBC has not been used in attestation-agent/Makefile in a while
so the build matrix runs the same loop five times.

The new parameter is "ATTESTER". Make all CI checks
(build/lint/unit test) to use that.

Signed-off-by: Mikko Ylinen <mikko.ylinen@intel.com>
  • Loading branch information
mythi authored and fitzthum committed Oct 14, 2024
1 parent dbae199 commit 25dbce5
Showing 1 changed file with 9 additions and 11 deletions.
20 changes: 9 additions & 11 deletions .github/workflows/aa_cc_kbc.yml
Original file line number Diff line number Diff line change
Expand Up @@ -35,12 +35,11 @@ jobs:
matrix:
rust:
- stable
kbc:
- cc_kbc
- cc_kbc_tdx
- cc_kbc_sgx
- cc_kbc_az_snp_vtpm
- cc_kbc_snp
attester:
- snp-attester
- tdx-attester
- az-snp-vtpm-attester
- az-tdx-vtpm-attester
steps:
- name: Code checkout
uses: actions/checkout@v4
Expand Down Expand Up @@ -68,19 +67,18 @@ jobs:
run: |
sudo apt-get update && sudo apt-get install -y protobuf-compiler
- name: Build and install with ${{ matrix.kbc }} feature
- name: Build and install with ${{ matrix.attester }} feature
run: |
make KBC=${{ matrix.kbc }} && make install
make ATTESTER=${{ matrix.attester }} && make install
- name: Run rust lint check
uses: actions-rs/cargo@v1
with:
command: clippy
# We are getting error in generated code due to derive_partial_eq_without_eq check, so ignore it for now
args: -p kbc --all-targets --features cc_kbc,all-attesters,rust-crypto -- -D warnings -A clippy::derive_partial_eq_without_eq
args: -p kbc --all-targets --features cc_kbc,${{ matrix.attester }},rust-crypto -- -D warnings

- name: Run cargo test
uses: actions-rs/cargo@v1
with:
command: test
args: --features cc_kbc,all-attesters,rust-crypto -p kbc
args: --features cc_kbc,${{ matrix.attester }},rust-crypto -p kbc

0 comments on commit 25dbce5

Please sign in to comment.