Title: fix(ci): run Build App on the larger runner in GitHub mode by TheodoreSpeaks · Pull Request #5826 · simstudioai/sim · GitHub
Open Graph Title: fix(ci): run Build App on the larger runner in GitHub mode by TheodoreSpeaks · Pull Request #5826 · simstudioai/sim
X Title: fix(ci): run Build App on the larger runner in GitHub mode by TheodoreSpeaks · Pull Request #5826 · simstudioai/sim
Description: Summary Follow-up to #5814. Tonight's live CI_PROVIDER=github run caught the one job I missed. The app is built twice: once inside Docker for the deployable image (Build AMD64 (app), already routed to linux-x64-8-core in #5814) and once outside Docker as a parallel check (Build App in test-build.yml). Same next build, same ~21.9 GB peak — but Build App was still on a free 16 GB runner and SIGKILLed: error: script "build" was terminated by signal SIGKILL (Forced quit) That failure blocked Migrate DB and Promote Images, so staging never deployed the merge commit. Lint and Test stays on free ubuntu-latest — it passed on GitHub-hosted with 13,589 tests and doesn't run the Next build. Why unconditional rather than fork-guarded test-build.yml runs on PRs including forks, so a paid runner here is exposed to fork traffic in a way the push-only image jobs aren't. That's acceptable: this repo requires maintainer approval for fork-PR workflows (verified — fork PRs #5801/#5791 sit at action_required), so nobody can trigger spend by opening a PR. An approved fork PR costs ~$0.15 instead of $0, and it executes no code it wasn't already going to execute on a free runner. Public IP is disabled on the runner, so GitHub's fixed-IP warning doesn't apply. A fork guard would have added a conditional and made outside contributors' PRs fail with a confusing OOM. Costs nothing in normal operation — Blacksmith is unchanged. Testing actionlint clean, bun run lint:check 19/19 Evidence from run 29877014793: Build AMD64 (app) succeeded on linux-x64-8-core while Build App SIGKILLed on ubuntu-latest in the same run — same build, only the runner differed Memory requirement measured directly: 21,883 MB peak on a 32 GB runner; free 16 GB runners fail at every heap ceiling (4096/6144/8192 all died at ~15.9 GB) Type of Change Bug fix Checklist Code follows project style guidelines Self-reviewed my changes Tests added/updated and passing No new warnings introduced I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)
Open Graph Description: Summary Follow-up to #5814. Tonight's live CI_PROVIDER=github run caught the one job I missed. The app is built twice: once inside Docker for the deployable image (Build AMD64 (app), already ro...
X Description: Summary Follow-up to #5814. Tonight's live CI_PROVIDER=github run caught the one job I missed. The app is built twice: once inside Docker for the deployable image (Build AMD64 (app), alread...
Opengraph URL: https://github.com/simstudioai/sim/pull/5826
X: @github
Domain: github.com
| route-pattern | /:user_id/:repository/pull/:id/files(.:format) |
| route-controller | pull_requests |
| route-action | files |
| fetch-nonce | v2:8f002b82-71b8-28e7-f7f8-9960228dec9f |
| current-catalog-service-hash | ae870bc5e265a340912cde392f23dad3671a0a881730ffdadd82f2f57d81641b |
| request-id | 9D5A:1D1C2C:400E3B:5A6D43:6A637B34 |
| html-safe-nonce | edf1f8095f067abccd35d3087a42e1e3105efc108b01f2d96db22124e2643fda |
| visitor-payload | eyJyZWZlcnJlciI6IiIsInJlcXVlc3RfaWQiOiI5RDVBOjFEMUMyQzo0MDBFM0I6NUE2RDQzOjZBNjM3QjM0IiwidmlzaXRvcl9pZCI6IjE1MTcwMjYxMDkwNzQ2NjgzNDEiLCJyZWdpb25fZWRnZSI6ImlhZCIsInJlZ2lvbl9yZW5kZXIiOiJpYWQifQ== |
| visitor-hmac | 79aa2942405c21d7e1c7376336c04ed594b88875cb79ebc9dd46eaeeef2e4b46 |
| hovercard-subject-tag | pull_request:4104010258 |
| github-keyboard-shortcuts | repository,pull-request-list,pull-request-conversation,pull-request-files-changed,copilot |
| google-site-verification | Apib7-x98H0j5cPqHWwSMm6dNU4GmODRoqxLiDzdx9I |
| octolytics-url | https://collector.github.com/github/collect |
| analytics-location | / |
| fb:app_id | 1401488693436528 |
| apple-itunes-app | app-id=1477376905, app-argument=https://github.com/simstudioai/sim/pull/5826/files |
| twitter:image | https://avatars.githubusercontent.com/u/19564703?s=400&v=4 |
| twitter:card | summary_large_image |
| og:image | https://avatars.githubusercontent.com/u/19564703?s=400&v=4 |
| og:image:alt | Summary Follow-up to #5814. Tonight's live CI_PROVIDER=github run caught the one job I missed. The app is built twice: once inside Docker for the deployable image (Build AMD64 (app), already ro... |
| og:site_name | GitHub |
| og:type | object |
| hostname | github.com |
| expected-hostname | github.com |
| None | 4c1de337de00c7969335f8866af41463301071379a3a15fba12e9b85aa6378d6 |
| turbo-cache-control | no-preview |
| diff-view | unified |
| go-import | github.com/simstudioai/sim git https://github.com/simstudioai/sim.git |
| octolytics-dimension-user_id | 199344406 |
| octolytics-dimension-user_login | simstudioai |
| octolytics-dimension-repository_id | 912559512 |
| octolytics-dimension-repository_nwo | simstudioai/sim |
| octolytics-dimension-repository_public | true |
| octolytics-dimension-repository_is_fork | false |
| octolytics-dimension-repository_network_root_id | 912559512 |
| octolytics-dimension-repository_network_root_nwo | simstudioai/sim |
| turbo-body-classes | logged-out env-production page-responsive |
| disable-turbo | true |
| browser-stats-url | https://api.github.com/_private/browser/stats |
| browser-errors-url | https://api.github.com/_private/browser/errors |
| release | a4df32552e4c631011bdcd4c81cc3d7408664a8b |
| ui-target | full |
| theme-color | #1e2327 |
| color-scheme | light dark |
Links:
Viewport: width=device-width