From 37ea0c6e85726f5fb530775c76d23aca6a713321 Mon Sep 17 00:00:00 2001 From: Valerie Pomerleau Date: Thu, 23 Apr 2026 22:37:32 -0700 Subject: [PATCH] chore(ci): bump Unit Test (PR) to xlarge to complete FXA-13473 mitigation Because: - The `Unit Test (PR)` job in CircleCI runs on `default: large` (8 GB RAM), which is not enough headroom once `nx run-many --parallel=2` spawns two packages' jest processes, each with `maxWorkers: 4`. packages/fxa-auth-server/lib/metrics/context.spec.ts uses `jest.resetModules()` + `require('./context')` in every test, which retains a large require tree (joi + fxa-shared + validators); under --coverage this regularly OOM-kills a jest worker with SIGKILL. - FXA-13473 tracks the root-cause refactor (unscheduled), and its interim mitigation was to bump the Unit Test jobs to `xlarge`. That bump landed for `Unit Test` (tagged deploy, line 1115) and `Unit Test (nightly)` (line 1244) but was not applied to `Unit Test (PR)` (line 926), so PR runs still hit the OOM. This commit: - Adds `resource_class: xlarge` to the `Unit Test (PR)` workflow entry, matching the tagged-deploy and nightly variants. Uses more CircleCI credits per PR build but eliminates the SIGKILL flake until FXA-13473 lands. --- .circleci/config.yml | 1 + 1 file changed, 1 insertion(+) diff --git a/.circleci/config.yml b/.circleci/config.yml index 396cc1fd0e4..ded68ade615 100644 --- a/.circleci/config.yml +++ b/.circleci/config.yml @@ -925,6 +925,7 @@ workflows: - fail-fast - unit-test: name: Unit Test (PR) + resource_class: xlarge nx_run: affected --base=main --head=$CIRCLE_SHA1 workflow: test_pull_request requires: