ci: increase artifact Testbox memory - #99820
Conversation
|
Merged via squash.
|
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: db4811256a
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| if: steps.dist-cache.outputs.cache-hit != 'true' | ||
| env: | ||
| NODE_OPTIONS: --max-old-space-size=12288 | ||
| NODE_OPTIONS: --max-old-space-size=16384 |
There was a problem hiding this comment.
Set the tsdown heap override for artifact builds
When this workflow misses the dist cache and reaches the failing tsdown phase, this NODE_OPTIONS increase still gets clamped back to 12288 MB: pnpm build:ci-artifacts runs scripts/tsdown-build.mjs, and that wrapper only raises its resolved heap through OPENCLAW_TSDOWN_MAX_OLD_SPACE_MB; inherited --max-old-space-size=16384 is normalized down by normalizeMaxOldSpaceSizeMb. The artifact Testbox will therefore keep using the old 12 GB tsdown heap in exactly the OOM scenario this commit is meant to fix, so set the tsdown-specific override as well or adjust the wrapper contract.
Useful? React with 👍 / 👎.
* ci: increase artifact Testbox memory * ci: increase artifact Testbox memory * ci: increase artifact Testbox memory
What Problem This Solves
The Blacksmith Build Artifacts Testbox repeatedly exhausted memory while running
tsdownon its 8-vCPU runner, blocking otherwise-green pull requests. Exact failures reproduced in runs 28695538053 and 28695176535.Why This Change Was Made
The canonical CI artifact build already runs successfully on
blacksmith-16vcpu-ubuntu-2404. This aligns the Testbox artifact build with that proven runner and raises its Node heap ceiling from 12 GB to 16 GB.User Impact
Maintainer artifact validation no longer depends on repeatedly retrying a memory-starved build lane.
Evidence
node scripts/run-vitest.mjs test/scripts/ci-workflow-guards.test.ts— 33 passed.git diff --check— passed.This change was developed and validated with AI assistance. No transcript is included.