Changes for s390x

BbolroC · Jun 13, 2024 · c331272 · c331272
1 parent d2790bd
commit c331272
Showing 1 changed file with 36 additions and 33 deletions.
diff --git a/.github/workflows/aa_basic.yml b/.github/workflows/aa_basic.yml
@@ -3,18 +3,19 @@ name: attestation-agent basic build and unit tests
 on:
   push:
     branches:
-      - "main"
-    paths:
-      - 'attestation-agent/**'
-      - '.github/workflows/aa_basic.yml'
-      - 'Cargo.toml'
-      - 'Cargo.lock'
-  pull_request:
-    paths:
-      - 'attestation-agent/**'
-      - '.github/workflows/aa_basic.yml'
-      - 'Cargo.toml'
-      - 'Cargo.lock'
+      - "final-check-s390x"
+#      - "main"
+#    paths:
+#      - 'attestation-agent/**'
+#      - '.github/workflows/aa_basic.yml'
+#      - 'Cargo.toml'
+#      - 'Cargo.lock'
+#  pull_request:
+#    paths:
+#      - 'attestation-agent/**'
+#      - '.github/workflows/aa_basic.yml'
+#      - 'Cargo.toml'
+#      - 'Cargo.lock'
   #create:
   workflow_dispatch:
 
@@ -35,13 +36,15 @@ jobs:
           - stable
         instance:
           - ubuntu-22.04
-          - s390x
+          - s390x-test
         include:
           - instance: ubuntu-22.04
             make_args: ""
+            cargo_test_opts: "--features openssl,rust-crypto,all-attesters,kbs,coco_as"
             cargo_lint_opts: "--workspace"
-          - instance: s390x
+          - instance: s390x-test
             make_args: "ATTESTER=se-attester TEE_PLATFORM=se"
+            cargo_test_opts: "--no-default-features --features openssl,se-attester,kbs,coco_as"
             cargo_lint_opts: "--no-default-features --features openssl,se-attester,kbs,coco_as -p attestation-agent -p attester -p coco_keyprovider -p kbc -p kbs_protocol -p crypto -p resource_uri"
     runs-on: ${{ matrix.instance }}
     steps:
@@ -87,6 +90,7 @@ jobs:
           eval make ttrpc=true ${MAKE_ARGS} && make install PREFIX=${HOME}/.local
         env:
           MAKE_ARGS: ${{ matrix.make_args }}
+        if: matrix.instance == 'ubuntu-22.04'
 
       - name: Musl build with all platform
         run: |
@@ -97,25 +101,24 @@ jobs:
         uses: actions-rs/cargo@v1
         with:
           command: test
-          args: --features openssl,rust-crypto,all-attesters,kbs,coco_as -p attestation-agent -p attester -p coco_keyprovider -p kbc -p kbs_protocol -p crypto -p resource_uri
-        if: matrix.instance == 'ubuntu-22.04'  # will be enabled after https://github.com/confidential-containers/trustee/pull/383
+          args: ${{ matrix.cargo_test_opts }} -p attestation-agent -p attester -p coco_keyprovider -p kbc -p kbs_protocol -p crypto -p resource_uri
 
-      - name: Run cargo fmt check
-        uses: actions-rs/cargo@v1
-        with:
-          command: fmt
-          args: --all -- --check
+#      - name: Run cargo fmt check
+#        uses: actions-rs/cargo@v1
+#        with:
+#          command: fmt
+#          args: --all -- --check
 
-      - 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: ${{ matrix.cargo_lint_opts }} -- -D warnings -A clippy::derive-partial-eq-without-eq
+#      - 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: ${{ matrix.cargo_lint_opts }} -- -D warnings -A clippy::derive-partial-eq-without-eq
 
-      - name: Take a post-action for self-hosted runner
-        if: always()
-        run: |
-          if [ -f ${HOME}/script/post_action.sh ]; then
-            ${HOME}/script/post_action.sh cc-guest-components
-          fi
+#      - name: Take a post-action for self-hosted runner
+#        if: always()
+#        run: |
+#          if [ -f ${HOME}/script/post_action.sh ]; then
+#            ${HOME}/script/post_action.sh cc-guest-components
+#          fi