From 2df6ebd210c3a362f445175ee752d7326aa88628 Mon Sep 17 00:00:00 2001 From: rcourtman <8825017+rcourtman@users.noreply.github.com> Date: Wed, 9 Sep 2026 09:22:27 +0100 Subject: [PATCH] ci: avoid duplicate E2E artifact uploads Playwright HTML reports already include screenshots, videos and traces. Upload only the separate JUnit XML alongside each report to retain machine-readable results without storing failure attachments twice. --- .github/workflows/test-e2e.yml | 14 ++++++++------ 1 file changed, 8 insertions(+), 6 deletions(-) diff --git a/.github/workflows/test-e2e.yml b/.github/workflows/test-e2e.yml index b400e9b64..f2aec42b3 100644 --- a/.github/workflows/test-e2e.yml +++ b/.github/workflows/test-e2e.yml @@ -203,12 +203,14 @@ jobs: path: tests/integration/playwright-report/ retention-days: 3 - - name: Upload test videos and screenshots + # The HTML report already contains screenshots, videos and traces. + # Keep only the separate machine-readable result here. + - name: Upload JUnit results if: failure() uses: actions/upload-artifact@043fb46d1a93c77aae656e7c1c64a875d1fc6a0a # v7.0.1 with: - name: test-failures-shard-${{ matrix.shard }} - path: tests/integration/test-results/ + name: junit-results-shard-${{ matrix.shard }} + path: tests/integration/test-results/junit.xml retention-days: 3 # continue-on-error keeps the job green on a probation failure, so @@ -221,12 +223,12 @@ jobs: path: tests/integration/playwright-report-probation/ retention-days: 3 - - name: Upload probation test videos and screenshots + - name: Upload probation JUnit results if: ${{ !cancelled() && steps.probation.outcome == 'failure' }} uses: actions/upload-artifact@043fb46d1a93c77aae656e7c1c64a875d1fc6a0a # v7.0.1 with: - name: test-failures-probation-shard-${{ matrix.shard }} - path: tests/integration/test-results-probation/ + name: junit-results-probation-shard-${{ matrix.shard }} + path: tests/integration/test-results-probation/junit.xml retention-days: 3 agent-registration: