[ci] upload nightly FPGA test results to gcloud bucket

Signed-off-by: Alice Ziuziakowska <[email protected]>

Author: ziuziakowska

.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

.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,6 +62,7 @@ jobs:
     steps:
       - uses: actions/checkout@v4
         with:
+          ref: ${{ inputs.branch }}
           fetch-depth: 0
       - name: Prepare environment
         uses: ./.github/actions/prepare-env
@@ -103,6 +112,8 @@ jobs:
         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() }}

.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