Skip to content

Commit 274500c

Browse files
authored
Merge pull request #10 from pylon-code/fix/2026-08-12-release-artifact-retention
ci: stop artifact storage from failing CI, and cap release artifact retention
2 parents e44d62b + b260397 commit 274500c

2 files changed

Lines changed: 18 additions & 0 deletions

File tree

.github/workflows/ci.yml

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -98,8 +98,16 @@ jobs:
9898
echo "Transfer budget report was not produced." >> "$GITHUB_STEP_SUMMARY"
9999
fi
100100
101+
# Non-fatal on purpose. This is a few KB of JSON whose only consumer is
102+
# thread-transfer-report.yml, which turns it into a PR comment. Account
103+
# artifact storage is a shared, org-wide resource that something else can
104+
# exhaust, and when it is exhausted every upload fails — so leaving this
105+
# step fatal lets an unrelated storage condition fail a run whose tests
106+
# passed, which is what happened to #9. Losing the transfer comment is the
107+
# correct way to degrade; blocking code review is not.
101108
- name: Upload thread transfer result
102109
if: always()
110+
continue-on-error: true
103111
uses: actions/upload-artifact@v7
104112
with:
105113
name: thread-transfer-results

.github/workflows/release.yml

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -706,19 +706,29 @@ jobs:
706706
mkdir -p "$target_dir"
707707
cp "$source_path" "$target_dir/$binary_name"
708708
709+
# These are intra-run handoff to `publish_release`, which downloads
710+
# `desktop-*` and attaches them to the GitHub Release — that Release is the
711+
# durable copy, not the artifact. At the repository default of 90 days a
712+
# daily nightly retains ~880 MB per run across the four platforms, which
713+
# reached ~14 GB and exhausted the account's artifact storage quota; once
714+
# that happens *every* upload in the org fails, reddening unrelated PRs.
715+
# A week is long enough to re-run a failed publish job against the same
716+
# build and short enough to cap steady state near 6 GB.
709717
- name: Upload build artifacts
710718
uses: actions/upload-artifact@v7
711719
with:
712720
name: desktop-${{ matrix.platform }}-${{ matrix.arch }}
713721
path: release-publish/*
714722
if-no-files-found: error
723+
retention-days: 7
715724

716725
- name: Upload resource monitor
717726
uses: actions/upload-artifact@v7
718727
with:
719728
name: resource-monitor-${{ matrix.resource_key }}
720729
path: resource-monitor-publish/${{ matrix.resource_key }}/*
721730
if-no-files-found: error
731+
retention-days: 7
722732

723733
publish_cli:
724734
name: Publish CLI to npm

0 commit comments

Comments
 (0)