Skip to content

Commit

Permalink
ci: use rust 1.82.0 for arm64
Browse files Browse the repository at this point in the history
rust on aarch64-unknown-linux-gnu has a bug which faces SIGSEGV
intermittently (rust-lang/rust#135867)
with 1.83.0 or later.
rust 1.82.0 will be used for arm64 only until the above issue is resolved.

Signed-off-by: Seunguk Shin <seunguk.shin@arm.com>
  • Loading branch information
Seunguk Shin committed Feb 5, 2025
1 parent 0a7051d commit 72323aa
Show file tree
Hide file tree
Showing 9 changed files with 76 additions and 4 deletions.
6 changes: 6 additions & 0 deletions .github/workflows/aa_basic.yml
Original file line number Diff line number Diff line change
Expand Up @@ -48,6 +48,12 @@ jobs:
- instance: ubuntu-24.04-arm
make_args: "ATTESTER=cca-attester TEE_PLATFORM=cca"
cargo_test_opts: "--no-default-features --features openssl,rust-crypto,passport,cca-attester,kbs,coco_as,ttrpc,grpc"
# TODO: remove rust: 1.82.0 and exclude when the following issue is resolved
# https://github.com/rust-lang/rust/issues/135867
rust: 1.82.0
exclude:
- instance: ubuntu-24.04-arm
rust: stable
runs-on: ${{ matrix.instance }}
steps:
- name: Code checkout
Expand Down
8 changes: 7 additions & 1 deletion .github/workflows/aa_cc_kbc.yml
Original file line number Diff line number Diff line change
Expand Up @@ -48,6 +48,12 @@ jobs:
attester: az-tdx-vtpm-attester
- instance: ubuntu-24.04-arm
attester: cca-attester
# TODO: remove rust: 1.82.0 and exclude when the following issue is resolved
# https://github.com/rust-lang/rust/issues/135867
rust: 1.82.0
exclude:
- instance: ubuntu-24.04-arm
rust: stable
runs-on: ${{ matrix.instance }}
steps:
- name: Code checkout
Expand All @@ -61,7 +67,7 @@ jobs:
profile: minimal
toolchain: ${{ matrix.rust }}
override: true
components: rustfmt
components: rustfmt, clippy

- uses: ./.github/actions/install-intel-dcap
with:
Expand Down
16 changes: 13 additions & 3 deletions .github/workflows/aa_crypto.yml
Original file line number Diff line number Diff line change
Expand Up @@ -34,6 +34,16 @@ jobs:
suites:
- rust-crypto
- openssl
rust:
- stable
# TODO: remove include and exclude when the following issue is resolved
# https://github.com/rust-lang/rust/issues/135867
- 1.82.0
exclude:
- instance: ubuntu-24.04
rust: 1.82.0
- instance: ubuntu-24.04-arm
rust: stable
runs-on: ${{ matrix.instance }}

steps:
Expand All @@ -42,13 +52,13 @@ jobs:
with:
fetch-depth: 1

- name: Install Rust toolchain (stable)
- name: Install Rust toolchain (${{ matrix.rust }})
uses: actions-rs/toolchain@v1
with:
profile: minimal
toolchain: stable
toolchain: ${{ matrix.rust }}
override: true
components: rustfmt
components: rustfmt, clippy

- name: Run rust fmt check
uses: actions-rs/cargo@v1
Expand Down
8 changes: 8 additions & 0 deletions .github/workflows/aa_sample_keyprovider.yml
Original file line number Diff line number Diff line change
Expand Up @@ -29,6 +29,14 @@ jobs:
instance:
- ubuntu-24.04
- ubuntu-24.04-arm
# TODO: remove include and exclude when the following issue is resolved
# https://github.com/rust-lang/rust/issues/135867
include:
- instance: ubuntu-24.04-arm
rust: 1.82.0
exclude:
- instance: ubuntu-24.04-arm
rust: stable
runs-on: ${{ matrix.instance }}

steps:
Expand Down
8 changes: 8 additions & 0 deletions .github/workflows/api-server-rest-basic.yml
Original file line number Diff line number Diff line change
Expand Up @@ -38,6 +38,14 @@ jobs:
- ubuntu-24.04-arm
rust:
- stable
# TODO: remove include and exclude when the following issue is resolved
# https://github.com/rust-lang/rust/issues/135867
include:
- instance: ubuntu-24.04-arm
rust: 1.82.0
exclude:
- instance: ubuntu-24.04-arm
rust: stable
runs-on: ${{ matrix.instance }}
steps:
- name: Code checkout
Expand Down
8 changes: 8 additions & 0 deletions .github/workflows/cdh_basic.yml
Original file line number Diff line number Diff line change
Expand Up @@ -38,6 +38,14 @@ jobs:
- ubuntu-24.04-arm
rust:
- stable
# TODO: remove include and exclude when the following issue is resolved
# https://github.com/rust-lang/rust/issues/135867
include:
- instance: ubuntu-24.04-arm
rust: 1.82.0
exclude:
- instance: ubuntu-24.04-arm
rust: stable
runs-on: ${{ matrix.instance }}
steps:
- name: Code checkout
Expand Down
10 changes: 10 additions & 0 deletions .github/workflows/image_rs_build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -38,6 +38,16 @@ jobs:
- ubuntu-24.04
- s390x
- ubuntu-24.04-arm
# TODO: remove include and exclude when the following issue is resolved
# https://github.com/rust-lang/rust/issues/135867
include:
- instance: ubuntu-24.04-arm
rust: 1.82.0
exclude:
- instance: ubuntu-24.04-arm
rust: stable
- instance: ubuntu-24.04-arm
rust: 1.83.0
runs-on: ${{ matrix.instance }}
steps:
- name: Code checkout
Expand Down
10 changes: 10 additions & 0 deletions .github/workflows/ocicrypt_rs_build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -34,6 +34,16 @@ jobs:
rust:
- 1.83.0
- stable
# TODO: remove include and exclude when the following issue is resolved
# https://github.com/rust-lang/rust/issues/135867
include:
- instance: ubuntu-24.04-arm
rust: 1.82.0
exclude:
- instance: ubuntu-24.04-arm
rust: stable
- instance: ubuntu-24.04-arm
rust: 1.83.0
runs-on: ${{ matrix.instance }}

steps:
Expand Down
6 changes: 6 additions & 0 deletions .github/workflows/trustee-attester.yml
Original file line number Diff line number Diff line change
Expand Up @@ -36,6 +36,12 @@ jobs:
cargo_test_opts: "-p kbs_protocol --bin trustee-attester --no-default-features --features background_check,passport,openssl,se-attester,bin"
- instance: ubuntu-24.04-arm
cargo_test_opts: "-p kbs_protocol --bin trustee-attester --no-default-features --features background_check,passport,openssl,cca-attester,bin"
# TODO: remove rust: 1.82.0 and exclude when the following issue is resolved
# https://github.com/rust-lang/rust/issues/135867
rust: 1.82.0
exclude:
- instance: ubuntu-24.04-arm
rust: stable
runs-on: ${{ matrix.instance }}
steps:
- name: Code checkout
Expand Down

0 comments on commit 72323aa

Please sign in to comment.