From a12a82a46f27b659d7fc79fac1798bae220fac06 Mon Sep 17 00:00:00 2001
From: rcourtman <courtmanr@gmail.com>
Date: Mon, 20 Jul 2026 09:54:57 +0100
Subject: [PATCH] Upload e2e artifacts only on failure with 3-day retention

Playwright HTML reports uploaded on every run at 30-day retention blew
the Actions storage quota (69.8 GB on 2026-07-20). Reports now upload
only when a shard fails, and all report/video/screenshot artifacts
keep a 3-day retention.
---
 .github/workflows/test-e2e.yml | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

diff --git a/.github/workflows/test-e2e.yml b/.github/workflows/test-e2e.yml
index 669e78a7df..0f97b5e61e 100644
--- a/.github/workflows/test-e2e.yml
+++ b/.github/workflows/test-e2e.yml
@@ -103,12 +103,12 @@ jobs:
           docker logs pulse-mock-github 2>&1 || echo "No pulse-mock-github container"
 
       - name: Upload Playwright report
-        if: always()
+        if: failure()
         uses: actions/upload-artifact@043fb46d1a93c77aae656e7c1c64a875d1fc6a0a # v7.0.1
         with:
           name: playwright-report-shard-${{ matrix.shard }}
           path: tests/integration/playwright-report/
-          retention-days: 30
+          retention-days: 3
 
       - name: Upload test videos and screenshots
         if: failure()
@@ -116,7 +116,7 @@ jobs:
         with:
           name: test-failures-shard-${{ matrix.shard }}
           path: tests/integration/test-results/
-          retention-days: 7
+          retention-days: 3
 
   agent-registration:
     name: Agent registration lifecycle
@@ -178,12 +178,12 @@ jobs:
         run: docker logs pulse-test-server 2>&1 || echo "No pulse-test-server container"
 
       - name: Upload Playwright report
-        if: always()
+        if: failure()
         uses: actions/upload-artifact@043fb46d1a93c77aae656e7c1c64a875d1fc6a0a # v7.0.1
         with:
           name: playwright-report-agent-registration
           path: tests/integration/playwright-report/
-          retention-days: 30
+          retention-days: 3
 
       - name: Upload test videos and screenshots
         if: failure()
@@ -191,7 +191,7 @@ jobs:
         with:
           name: test-failures-agent-registration
           path: tests/integration/test-results/
-          retention-days: 7
+          retention-days: 3
 
   e2e-verdict:
     name: E2E verdict