Skip to content

Commit 94db52d

Browse files
authored
Fix Workflow summaries for ARM workflows (#12707)
1 parent 8b11e74 commit 94db52d

File tree

1 file changed

+13
-21
lines changed

1 file changed

+13
-21
lines changed

.github/workflows/aws-tests.yml

Lines changed: 13 additions & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -447,12 +447,12 @@ jobs:
447447
name: Publish Test Results
448448
strategy:
449449
matrix:
450-
runner:
451-
- ubuntu-latest
452-
- ubuntu-24.04-arm
450+
arch:
451+
- amd64
452+
- arm64
453453
exclude:
454454
# skip the ARM integration tests in case we are not on the master and not on the upgrade-dependencies branch and forceARMTests is not set to true
455-
- runner: ${{ (github.ref != 'refs/heads/master' && github.ref != 'upgrade-dependencies' && inputs.forceARMTests == false) && 'ubuntu-24.04-arm' || ''}}
455+
- arch: ${{ (github.ref != 'refs/heads/master' && github.ref != 'upgrade-dependencies' && inputs.forceARMTests == false) && 'arm64' || ''}}
456456
needs:
457457
- test-integration
458458
- test-bootstrap
@@ -465,28 +465,24 @@ jobs:
465465
# execute on success or failure, but not if the workflow is cancelled or any of the dependencies has been skipped
466466
if: always() && !cancelled() && !contains(needs.*.result, 'skipped')
467467
steps:
468-
- name: Determine Runner Architecture
469-
shell: bash
470-
run: echo "PLATFORM=${{ (runner.arch == 'X64' && 'amd64') || (runner.arch == 'ARM64' && 'arm64') || '' }}" >> $GITHUB_ENV
471-
472468
- name: Download Bootstrap Artifacts
473469
uses: actions/download-artifact@v4
474-
if: ${{ env.PLATFORM == 'amd64' }}
470+
if: ${{ matrix.arch == 'amd64' }}
475471
with:
476472
pattern: test-results-bootstrap
477473

478474
- name: Download Integration Artifacts
479475
uses: actions/download-artifact@v4
480476
with:
481-
pattern: test-results-integration-${{ env.PLATFORM }}-*
477+
pattern: test-results-integration-${{ matrix.arch }}-*
482478

483479
- name: Publish Bootstrap and Integration Test Results
484480
uses: EnricoMi/publish-unit-test-result-action@v2
485481
if: success() || failure()
486482
with:
487483
files: |
488484
**/pytest-junit-*.xml
489-
check_name: "Test Results (${{ env.PLATFORM }}${{ inputs.testAWSAccountId != '000000000000' && ', MA/MR' || ''}}) - Integration${{ env.PLATFORM == 'amd64' && ', Bootstrap' || ''}}"
485+
check_name: "Test Results (${{ matrix.arch }}${{ inputs.testAWSAccountId != '000000000000' && ', MA/MR' || ''}}) - Integration${{ matrix.arch == 'amd64' && ', Bootstrap' || ''}}"
490486
test_file_prefix: "-/opt/code/localstack/"
491487
action_fail_on_inconclusive: true
492488

@@ -571,12 +567,12 @@ jobs:
571567
name: Publish Acceptance Test Results
572568
strategy:
573569
matrix:
574-
runner:
575-
- ubuntu-latest
576-
- ubuntu-24.04-arm
570+
arch:
571+
- amd64
572+
- arm64
577573
exclude:
578574
# skip the ARM integration tests in case we are not on the master and not on the upgrade-dependencies branch and forceARMTests is not set to true
579-
- runner: ${{ (github.ref != 'refs/heads/master' && github.ref != 'upgrade-dependencies' && inputs.forceARMTests == false) && 'ubuntu-24.04-arm' || ''}}
575+
- arch: ${{ (github.ref != 'refs/heads/master' && github.ref != 'upgrade-dependencies' && inputs.forceARMTests == false) && 'arm64' || ''}}
580576
needs:
581577
- test-acceptance
582578
runs-on: ubuntu-latest
@@ -588,22 +584,18 @@ jobs:
588584
# execute on success or failure, but not if the workflow is cancelled or any of the dependencies has been skipped
589585
if: always() && !cancelled() && !contains(needs.*.result, 'skipped')
590586
steps:
591-
- name: Determine Runner Architecture
592-
shell: bash
593-
run: echo "PLATFORM=${{ (runner.arch == 'X64' && 'amd64') || (runner.arch == 'ARM64' && 'arm64') || '' }}" >> $GITHUB_ENV
594-
595587
- name: Download Acceptance Artifacts
596588
uses: actions/download-artifact@v4
597589
with:
598-
pattern: test-results-acceptance-${{ env.PLATFORM }}
590+
pattern: test-results-acceptance-${{ matrix.arch }}
599591

600592
- name: Publish Acceptance Test Results
601593
uses: EnricoMi/publish-unit-test-result-action@v2
602594
if: success() || failure()
603595
with:
604596
files: |
605597
**/pytest-junit-*.xml
606-
check_name: "Test Results (${{ env.PLATFORM }}${{ inputs.testAWSAccountId != '000000000000' && ', MA/MR' || ''}}) - Acceptance"
598+
check_name: "Test Results (${{ matrix.arch }}${{ inputs.testAWSAccountId != '000000000000' && ', MA/MR' || ''}}) - Acceptance"
607599
test_file_prefix: "-/opt/code/localstack/"
608600
action_fail_on_inconclusive: true
609601

0 commit comments

Comments
 (0)