Skip to content

Commit 28e9332

Browse files
ci(deploy): cache Go modules via api/go.sum (fix cold-download fragility) (#227)
deploy.yml checks the api repo out into ./api, so setup-go's default go.sum lookup at the checkout root misses → the module cache is never restored and every deploy cold-downloads the full dep tree (~2 min). On 2026-06-03 a cold-download run was SIGTERM'd by a transient runner blip and the deploy only landed on a warm re-run. Point cache-dependency-path at api/go.sum so deps restore from cache, shrinking the download window + the transient-failure exposure. (ci.yml checks api out at the root, so its cache already works — this is deploy.yml-specific.) No code/behavior change. Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
1 parent c48d17a commit 28e9332

1 file changed

Lines changed: 8 additions & 0 deletions

File tree

.github/workflows/deploy.yml

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -135,6 +135,14 @@ jobs:
135135
uses: actions/setup-go@v6
136136
with:
137137
go-version: '1.25'
138+
# This workflow checks the api repo out into ./api (path: api above),
139+
# so setup-go's default go.sum lookup at the checkout root misses and
140+
# the module cache is never restored — every run cold-downloads the
141+
# full dep tree (~2 min), leaving the test step exposed to transient
142+
# runner blips (seen 2026-06-03: a cold-download run SIGTERM'd and the
143+
# deploy only landed on a warm re-run). Point the cache key at the real
144+
# go.sum so deps are restored from cache.
145+
cache-dependency-path: api/go.sum
138146

139147
- name: Stage sibling repos for go.mod replace (../common, ../proto)
140148
# The api repo's go.mod uses `replace instant.dev/common => ../common`

0 commit comments

Comments
 (0)