Skip to content

ci(codeql): cap Analyze (c-cpp) footprint on self-hosted 905 (fix co-OOM with ASan) - #611

Merged
frstrtr merged 1 commit into
masterfrom
ci-steward/codeql-905-oom
Jul 3, 2026
Merged

ci(codeql): cap Analyze (c-cpp) footprint on self-hosted 905 (fix co-OOM with ASan)#611
frstrtr merged 1 commit into
masterfrom
ci-steward/codeql-905-oom

Conversation

@frstrtr

@frstrtr frstrtr commented Jul 3, 2026

Copy link
Copy Markdown
Owner

Runner-side fix, not a code finding — do NOT change PR-author code.

Problem

The only red on DASH PRs #583/#571/#586 is CodeQL Analyze (c-cpp). The identical code compiles green under 4+ other toolchains (x86_64, ASan+UBSan, all COIN_* matrices, Windows, macOS, coin smokes). Root cause is the shared self-hosted 905 box, not the code:

  • Build for CodeQL analysis used -j$(nproc) (32-way).
  • Perform CodeQL Analysis auto-sized run-queries to --ram=60466 --threads=32 — nearly the whole box.

When CodeQL ran concurrently with an ASan job on 905 both co-OOMed (07-02 exit-137; #583 job 84734661452 step 10/11 conclusion=null kill).

Fix

Cap self-hosted only, leaving headroom for a concurrent ASan job:

  • build -j8
  • analyze threads: 12, ram: 20000 MB

github-hosted (fork) runs keep auto defaults (threads/ram: 0, -j$(nproc)) via the same fork-gate expression as PR #439. Single file, no author-code change.

Overlaps #439 self-hosted-CI oversubscription work — this is the same 905-box root cause folded into the CodeQL lane.

…osted 905

CodeQL was auto-sizing run-queries to --ram=60466 --threads=32 (nearly the
whole 905 box) and building with -j$(nproc). When it ran concurrently with
an ASan job on the shared self-hosted runner both jobs co-OOMed (exit-137 /
conclusion=null kill), leaving Analyze (c-cpp) as the lone red on DASH PRs
while every other toolchain compiled the identical code green.

Cap self-hosted only: build -j8, analyze threads=12 ram=20000MB, leaving
headroom for a concurrent ASan job. github-hosted (fork) runs keep auto
defaults (threads/ram=0, -j$(nproc)).
@frstrtr
frstrtr merged commit a57d20e into master Jul 3, 2026
21 checks passed
@frstrtr
frstrtr deleted the ci-steward/codeql-905-oom branch July 3, 2026 08:29
frstrtr added a commit that referenced this pull request Jul 3, 2026
…racker-leaf — fire canary against capped base
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant