Skip to content

fix(ci): count physical cores and report both capacity lanes - #686

Merged
zackees merged 1 commit into
mainfrom
fix/672-ci-capacity-report
Oct 18, 2026
Merged

zackees merged 1 commit into
mainfrom
fix/672-ci-capacity-report

Conversation

@zackees

@zackees zackees commented Oct 18, 2026

Copy link
Copy Markdown
Owner

Closes #672.

  • Set the default CI admission limit from physical cores. Linux counts package/core pairs in the process's allowed CPU set; macOS reads hw.physicalcpu; unknown topology or CPU quota falls back to half the available logical threads.
  • Report task slots and per-slot CPU quota separately from CI admission and effective shared-engine CPU/memory limits in bosn jobs and bosn ci doctor, including the 8-CPU default ceiling or [engine] cpus source.
  • Read frozen limits from a running machine-claimed engine even before this daemon leases it; record the CPU source on new claims, and label older claims' source unknown.
  • Update schema, protocol fingerprint, docs, and settled capacity policy.

RED: the focused 8-physical/16-logical topology test failed before implementation. GREEN: isolated Rust suite passed before review fixes; review regression then passed in the local gate, which passed both rust and tests lanes on this tree. ./lint and strict Clippy passed. One /clud-review reviewer found two reporting accuracy issues; both were fixed and rechecked clear.

Local-Gate: v1 tree=4d7d68c089502f1039a9dbeb8b730ec45adf164b secs=260 lanes=rust:run,tests:run
Ci-Attestation: {"at":1792294136,"gate":"rust/all/fmt","host":"linux-x86_64","key":"2e8acf938b137c3d0781da9a2c2051213123fd0cbcf0aa1aebf69b12fcfb1275","lane":"rust","parents":["f03038c220dd1d0638015f78a958676bee2e6c9a"],"secs":260,"stamp":"339653acab12556c1cc5f54fa236aa58","tree":"4d7d68c089502f1039a9dbeb8b730ec45adf164b","v":1,"via":"run"}
Ci-Attestation: {"at":1792294136,"gate":"rust/all/boundary","host":"linux-x86_64","key":"2e8acf938b137c3d0781da9a2c2051213123fd0cbcf0aa1aebf69b12fcfb1275","lane":"rust","parents":["f03038c220dd1d0638015f78a958676bee2e6c9a"],"secs":260,"stamp":"24eb846644569199515b9f20898b0e17","tree":"4d7d68c089502f1039a9dbeb8b730ec45adf164b","v":1,"via":"run"}
Ci-Attestation: {"at":1792294136,"gate":"rust/x86_64-unknown-linux-gnu/clippy","host":"linux-x86_64","key":"2e8acf938b137c3d0781da9a2c2051213123fd0cbcf0aa1aebf69b12fcfb1275","lane":"rust","parents":["f03038c220dd1d0638015f78a958676bee2e6c9a"],"secs":260,"stamp":"e42ccec7301ab123afed72c00a03930e","tree":"4d7d68c089502f1039a9dbeb8b730ec45adf164b","v":1,"via":"run"}
Ci-Attestation: {"at":1792294136,"gate":"rust/x86_64-unknown-linux-gnu/test","host":"linux-x86_64","key":"2e8acf938b137c3d0781da9a2c2051213123fd0cbcf0aa1aebf69b12fcfb1275","lane":"rust","parents":["f03038c220dd1d0638015f78a958676bee2e6c9a"],"secs":260,"stamp":"c3c3dd9fb391bbcbe880c988c7dae40f","tree":"4d7d68c089502f1039a9dbeb8b730ec45adf164b","v":1,"via":"run"}
Ci-Attestation: {"at":1792294136,"gate":"python/all/static","host":"linux-x86_64","key":"0f0ff80cd4101c860d3ecc1c1bcc08fb0307aabac1d71621d0c354b38063200d","lane":"tests","parents":["f03038c220dd1d0638015f78a958676bee2e6c9a"],"secs":243,"stamp":"5cda80f7bcc8ae854457024f2bad9bcf","tree":"4d7d68c089502f1039a9dbeb8b730ec45adf164b","v":1,"via":"run"}
Ci-Attestation: {"at":1792294136,"gate":"general/all/ci-policy","host":"linux-x86_64","key":"0f0ff80cd4101c860d3ecc1c1bcc08fb0307aabac1d71621d0c354b38063200d","lane":"tests","parents":["f03038c220dd1d0638015f78a958676bee2e6c9a"],"secs":243,"stamp":"0c9f91b859292b13c2fe89b9cf244ea6","tree":"4d7d68c089502f1039a9dbeb8b730ec45adf164b","v":1,"via":"run"}
Ci-Attestation: {"at":1792294136,"gate":"python/x86_64-unknown-linux-gnu/test","host":"linux-x86_64","key":"0f0ff80cd4101c860d3ecc1c1bcc08fb0307aabac1d71621d0c354b38063200d","lane":"tests","parents":["f03038c220dd1d0638015f78a958676bee2e6c9a"],"secs":243,"stamp":"205811a20a3cc41217838b63cafbc8ab","tree":"4d7d68c089502f1039a9dbeb8b730ec45adf164b","v":1,"via":"run"}
@zackees
zackees merged commit 626b457 into main Oct 18, 2026
14 checks passed
@zackees
zackees deleted the fix/672-ci-capacity-report branch October 18, 2026 03:30
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.

fix(capacity): CI admission default counts logical threads against its physical-core doc; show the 8-CPU engine cap in bosn jobs

1 participant