Skip to content

Commit 91ada48

Browse files
committed
Scope tarball retention cap to the intra-run build path
The previous commit put retention-days: 1 in the shared build-dist-tarball action, but create-release.yml uses the same action to produce the "Deb Zip" / "Deb Arm64 Zip" release tarballs, and test-smokes-built.yml re-tests older create-release runs via source=nightly / run-id. A 1-day cap there expires the "Deb Zip" after a day, so re-testing a run older than that resolves has-linux=false and silently skips the Linux smoke/playwright/ff-matrix legs while Windows/Mac artifacts (uploaded elsewhere, still 90 days) keep the run green. Make retention a composite-action input defaulting to the repo default, and cap it to 1 day only at the test-smokes-built build-mode call site, the sole true intra-run transport. compression-level: 0 stays in the action - every caller uploads an already-gzipped .tar.gz.
1 parent a698ae6 commit 91ada48

2 files changed

Lines changed: 8 additions & 3 deletions

File tree

‎.github/actions/build-dist-tarball/action.yml‎

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -19,6 +19,10 @@ inputs:
1919
artifact-name:
2020
description: "Workflow artifact name to upload the tarball as"
2121
required: true
22+
retention-days:
23+
description: "Artifact retention in days (empty = repo default; upload-artifact ignores an empty value)"
24+
required: false
25+
default: ""
2226
runs:
2327
using: "composite"
2428
steps:
@@ -51,8 +55,6 @@ runs:
5155
with:
5256
name: ${{ inputs.artifact-name }}
5357
path: ./package/${{ inputs.tarball-name }}
54-
# intra-run transport only: legs of the same run download it minutes
55-
# after upload; nothing reads it after the run
56-
retention-days: 1
58+
retention-days: ${{ inputs.retention-days }}
5759
# the tarball is already gzipped - re-compressing wastes CPU
5860
compression-level: 0

‎.github/workflows/test-smokes-built.yml‎

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -121,6 +121,9 @@ jobs:
121121
arch: amd64
122122
tarball-name: built-quarto-linux-amd64.tar.gz
123123
artifact-name: built-quarto-linux-amd64
124+
# intra-run transport only: the smoke/playwright/ff-matrix legs of
125+
# this run download it minutes later; nothing reads it afterward
126+
retention-days: 1
124127

125128
run-smokes-artifact:
126129
name: Smoke tests against built artifact

0 commit comments

Comments
 (0)