Skip to content

ci: reduce sanitizer/valgrind worker count to avoid runner OOM#9280

Merged
daywalker90 merged 1 commit into
ElementsProject:masterfrom
nGoline:ci-oom-runner-fix-9279
Jul 6, 2026
Merged

ci: reduce sanitizer/valgrind worker count to avoid runner OOM#9280
daywalker90 merged 1 commit into
ElementsProject:masterfrom
nGoline:ci-oom-runner-fix-9279

Conversation

@nGoline

@nGoline nGoline commented Jul 3, 2026

Copy link
Copy Markdown
Collaborator

Closes #9279

  • The ASan/UBSan and Valgrind shards intermittently have their hosted runner killed mid-run with "The runner has received a shutdown signal", surfaced as a mass test cancellation; the VM itself is being terminated, most likely from memory exhaustion.
  • -n $(nproc)+1 runs 5 parallel node-clusters and ASan roughly triples RSS, peaking past the 16GB hosted-runner ceiling on heavy shards (deterministic repro at ~10% on ASan/UBSan 3/6 due to the fixed --test-group-random-seed=42).
  • Drop integration-valgrind and integration-sanitizers to -n $(nproc) (one fewer worker) to lower peak memory.
  • Add an if: failure() step that dumps OOM-related dmesg and kernel-log lines so the next failure records the actual cause.
  • Direction acked by @cdecker in CI: ASan/Valgrind shards cancelled mid-run — runner receives shutdown signal (likely OOM on 16GB hosted runner) #9279.

The ASan/UBSan and Valgrind shards intermittently have their hosted
runner killed mid-run with "The runner has received a shutdown signal",
surfaced as a mass test cancellation.  The VM itself is being
terminated, most likely from memory exhaustion: `-n $(nproc)+1` runs 5
parallel node-clusters and ASan roughly triples RSS, peaking past the
16GB hosted-runner ceiling on heavy shards (deterministic repro at ~10%
on ASan/UBSan 3/6 due to the fixed --test-group-random-seed=42).

Drop these two jobs to `-n $(nproc)` (one fewer worker) to lower peak
memory, and add an `if: failure()` step that dumps OOM-related dmesg and
kernel-log lines so the next failure records the actual cause.

Changelog-None
@nGoline nGoline requested a review from cdecker July 3, 2026 20:19
@nGoline nGoline added this to the v26.09 milestone Jul 3, 2026
@nGoline nGoline added the Status::Ready for Review The work has been completed and is now awaiting evaluation or approval. label Jul 3, 2026
@daywalker90

Copy link
Copy Markdown
Collaborator

Thank you for this, this was getting annoying.

@daywalker90 daywalker90 merged commit 0da8fa2 into ElementsProject:master Jul 6, 2026
86 of 89 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Status::Ready for Review The work has been completed and is now awaiting evaluation or approval.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

CI: ASan/Valgrind shards cancelled mid-run — runner receives shutdown signal (likely OOM on 16GB hosted runner)

2 participants