[ci] upload nightly FPGA test results to gcloud bucket

ziuziakowska · ziuziakowska · commit 0547373f957d · 2025-11-26T15:44:47.000Z
Signed-off-by: Alice Ziuziakowska &lt;a.ziuziakowska@lowrisc.org&gt;
diff --git a/.github/actions/publish-bazel-test-results/action.yml b/.github/actions/publish-bazel-test-results/action.yml
@@ -7,14 +7,17 @@ description: Merge Bazel test results and publish the report
 
 inputs:
   merged-results:
-    description: Path to place merged JUnit report
+    description: Path to place merged JUnit report.
     default: test_results.xml
   artifact-name:
     description: Name of uploaded artifact. Leave empty to skip upload.
     default: ''
-  bucket-destination:
-    description: GCP bucket destination to upload report to.
-    default: ''
+  upload-to-bucket:
+    description: Whether to upload results to the Google Cloud Bucket.
+    default: "false"
+  branch:
+    description: Branch name.
+    default: "${{ github.ref }}"
 
 runs:
   using: composite
@@ -45,10 +48,12 @@ runs:
         overwrite: true
 
     - name: Upload report to Google Cloud
-      if: inputs.bucket-destination != ''
+      if: inputs.upload-to-bucket
       shell: bash
       run: |
-        gcloud storage cp "${{ inputs.merged-results }}" "gs://${{ inputs.bucket-destination }}"
+        BUCKET_PATH=opentitan-test-results/job/${{ github.job }}/branch/${{ inputs.branch }}/$(date +%Y-%m-%d-%H%M%S)_test_results.xml
+        echo "BUCKET_PATH=$BUCKET_PATH" >> $GITHUB_OUTPUT
+        gcloud storage cp "${{ inputs.merged-results }}" "gs://${BUCKET_PATH}"
 
     - name: Publish job summary
       uses: mikepenz/action-junit-report@ec3a351c13e080dc4fa94c49ab7ad5bf778a9668 # v5
diff --git a/.github/workflows/fpga.yml b/.github/workflows/fpga.yml
@@ -18,6 +18,10 @@ on:
         required: true
         type: string
         description: FPGA board to use
+      branch:
+        default: "${{ github.ref }}"
+        type: string
+        description: Branch to checkout
       interface:
         required: true
         type: string
@@ -45,6 +49,10 @@ on:
         default: true
         type: boolean
         description: Skip tests that previously passed and were cached by Bazel
+      upload_to_bucket:
+        default: false
+        type: boolean
+        description: Whether to upload FPGA results to Google Cloud Bucket
 
 jobs:
   fpga:
@@ -54,7 +62,9 @@ jobs:
     steps:
       - uses: actions/checkout@v4
         with:
+          ref: ${{ inputs.branch }}
           fetch-depth: 0
+
       - name: Prepare environment
         uses: ./.github/actions/prepare-env
         with:
@@ -73,36 +83,46 @@ jobs:
 
       - name: Execute tests
         run: |
-          . util/build_consts.sh
-          module load "xilinx/vivado/${{ inputs.vivado_version }}"
+          echo ${{ inputs.branch }}
+          # TODO: uncomment all this
+          # . util/build_consts.sh
+          # module load "xilinx/vivado/${{ inputs.vivado_version }}"
 
-          tag_filters="${{ inputs.tag_filters }}"
-          if ${{ inputs.add_default_filters }}; then
-            tag_filters+=",-manual,-broken,-slow_test,-skip_in_ci"
-          fi
+          # tag_filters="${{ inputs.tag_filters }}"
+          # if ${{ inputs.add_default_filters }}; then
+          #   tag_filters+=",-manual,-broken,-slow_test,-skip_in_ci"
+          # fi
 
-          # Execute a query to find all targets that match the test tags and store them in a file.
-          ci/scripts/run-bazel-test-query.sh \
-            target_pattern_file.txt \
-            "$tag_filters" \
-            ${{ inputs.targets }}
+          # # Execute a query to find all targets that match the test tags and store them in a file.
+          # ci/scripts/run-bazel-test-query.sh \
+          #   target_pattern_file.txt \
+          #   "$tag_filters" \
+          #   ${{ inputs.targets }}
 
-          cache_test_results="--cache_test_results=${{ inputs.cache_test_results }}"
+          # cache_test_results="--cache_test_results=${{ inputs.cache_test_results }}"
 
-          # Run FPGA tests
-          if [[ -s target_pattern_file.txt ]]; then
-            ci/scripts/run-fpga-tests.sh "${{ inputs.interface }}" target_pattern_file.txt "$cache_test_results" || {
-              res=$?; echo "To reproduce failures locally, follow the instructions at https://opentitan.org/book/doc/getting_started/setup_fpga.html#reproducing-fpga-ci-failures-locally"; exit "${res}";
-            }
-          else
-            echo "No tests to run after filtering"
-          fi
+          # # Run FPGA tests
+          # if [[ -s target_pattern_file.txt ]]; then
+          #   ci/scripts/run-fpga-tests.sh "${{ inputs.interface }}" target_pattern_file.txt "$cache_test_results" || {
+          #     res=$?; echo "To reproduce failures locally, follow the instructions at https://opentitan.org/book/doc/getting_started/setup_fpga.html#reproducing-fpga-ci-failures-locally"; exit "${res}";
+          #   }
+          # else
+          #   echo "No tests to run after filtering"
+          # fi
+
+      - uses: actions/checkout@v4
+        with:
+          ref: ${{ github.ref }}
+          path: "upload-branch"
+          fetch-depth: 1
 
       - name: Publish Bazel test results
-        uses: ./.github/actions/publish-bazel-test-results
+        uses: ./upload-branch/.github/actions/publish-bazel-test-results
         if: ${{ !cancelled() }}
         with:
           artifact-name: ${{ inputs.job_name }}-test-results
+          upload-to-bucket: ${{ inputs.upload_to_bucket }}
+          branch: ${{ inputs.branch }}
 
       - name: Upload target pattern file
         if: ${{ !cancelled() }}
diff --git a/.github/workflows/nightly.yml b/.github/workflows/nightly.yml
@@ -59,6 +59,8 @@ jobs:
       tag_filters: cw310_test_rom,-broken,-manual
       add_default_filters: false
       cache_test_results: false
+      upload_to_bucket: true
+      branch: "${{ inputs.branch }}"
 
   execute_fpga_cw310_rom_tests:
     name: CW310 ROM Tests
@@ -74,6 +76,8 @@ jobs:
       add_default_filters: false
       cache_test_results: false
       timeout: 240
+      upload_to_bucket: true
+      branch: "${{ inputs.branch }}"
 
   execute_fpga_cw310_rom_ext_tests:
     name: CW310 ROM_EXT Tests
@@ -88,6 +92,8 @@ jobs:
       tag_filters: cw310_rom_ext,-broken,-manual
       add_default_filters: false
       cache_test_results: false
+      upload_to_bucket: true
+      branch: "${{ inputs.branch }}"
 
   execute_fpga_cw310_sival_tests:
     name: CW310 SiVal Tests
@@ -102,6 +108,8 @@ jobs:
       tag_filters: cw310_sival,-broken,-manual
       add_default_filters: false
       cache_test_results: false
+      upload_to_bucket: true
+      branch: "${{ inputs.branch }}"
 
   execute_fpga_cw310_sival_rom_ext_tests:
     name: CW310 SiVal ROM_EXT Tests
@@ -116,6 +124,8 @@ jobs:
       tag_filters: cw310_sival_rom_ext,-broken,-manual
       add_default_filters: false
       cache_test_results: false
+      upload_to_bucket: true
+      branch: "${{ inputs.branch }}"
 
   execute_fpga_cw310_bob_tests:
     name: CW310 BoB (SPI and I2C) Tests
@@ -131,6 +141,8 @@ jobs:
       tag_filters: cw310_sival_rom_ext_no_hyper,-broken,-manual
       add_default_filters: false
       cache_test_results: false
+      upload_to_bucket: true
+      branch: "${{ inputs.branch }}"
 
   execute_fpga_cw340_test_rom_tests:
     name: CW340 SiVal Test ROM Tests
@@ -145,6 +157,8 @@ jobs:
       tag_filters: cw340_test_rom,-broken,-manual
       add_default_filters: false
       cache_test_results: false
+      upload_to_bucket: true
+      branch: "${{ inputs.branch }}"
 
   execute_fpga_cw340_rom_tests:
     name: CW340 SiVal ROM Tests
@@ -159,6 +173,8 @@ jobs:
       tag_filters: cw340_rom_with_fake_keys,-broken,-manual
       add_default_filters: false
       cache_test_results: false
+      upload_to_bucket: true
+      branch: "${{ inputs.branch }}"
 
   execute_fpga_cw340_rom_ext_tests:
     name: CW340 SiVal ROM_EXT Tests
@@ -173,6 +189,8 @@ jobs:
       tag_filters: cw340_rom_ext,-broken,-manual
       add_default_filters: false
       cache_test_results: false
+      upload_to_bucket: true
+      branch: "${{ inputs.branch }}"
 
   execute_fpga_cw340_sival_tests:
     name: CW340 SiVal Tests
@@ -187,6 +205,8 @@ jobs:
       tag_filters: cw340_sival,-broken,-manual
       add_default_filters: false
       cache_test_results: false
+      upload_to_bucket: true
+      branch: "${{ inputs.branch }}"
 
   execute_fpga_cw340_sival_rom_ext_tests:
     name: CW340 SiVal ROM_EXT Tests
@@ -202,6 +222,8 @@ jobs:
       add_default_filters: false
       cache_test_results: false
       timeout: 90
+      upload_to_bucket: true
+      branch: "${{ inputs.branch }}"
 
   slow_otbn_crypto_tests:
     name: Slow OTBN Crypto Tests
