Skip to content

fix(backend): run the test suite in forked processes so CI stops being OOM-killed #975

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
Makisuo merged 1 commit into from
Sep 22, 2026
Merged

fix(backend): run the test suite in forked processes so CI stops being OOM-killed #975

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
13 changes: 6 additions & 7 deletions packages/backend/vitest.config.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4,13 +4,12 @@ export default defineConfig({
test: {
environment: "node",
include: ["src/**/*.test.ts"],
// Worker threads rather than the default forked processes. Measured on the
// full suite at CI's 4 workers: 42.4s vs 45.2s wall, and 248s vs 292s of
// user CPU — the saving is process startup and the per-worker module
// registry, which threads share. `isolate` stays on: turning it off cut
// import time by 60% but pushed test time up further and came out slower
// overall (34.7s vs 29.9s at full local parallelism).
pool: "threads",
// Forked processes, not worker threads. Threads measured ~3s faster on the
// full suite, but on Linux CI the one process hosting every thread grew past
// the runner's 16 GB over ~140 files of PGlite instances and was SIGKILLed
// (exit 137, main on 2026-09-21); a fork hands its memory back at exit.
// `isolate` stays on: turning it off measured slower overall.
pool: "forks",
// Builds the post-migration PGlite data directory that createTestDb boots
// from. See test/pglite-snapshot.ts — without it every test pays a full
// initdb inside WASM.
Expand Down
Loading