Skip to content

docs(diary): record CI xdist parallelization - #48

Merged
ManSio merged 1 commit into
mainfrom
docs/ci-xdist-diary
Sep 26, 2026
Merged

ManSio merged 1 commit into
mainfrom
docs/ci-xdist-diary

Conversation

@ManSio

@ManSio ManSio commented Sep 26, 2026

Copy link
Copy Markdown
Owner

What

Records the CI/test parallelization work (#45/#46/#47) in AGENT_DIARY.md and EXPERIMENTS_LOG.md.

Correction (important)

EXPERIMENTS_LOG Exp 25 / diary E13 previously concluded "xdist gives only ~15%, not worth it" and "local xdist disabled (llama conflict)". That verdict was measured with -n 4 on a many-core machine. With -n auto:

  • local: 197.1s -> 71.4s (x2.76; +coverage 86.4s), 1865 passed / 0 failed
  • CI: ubuntu 13m44s -> 1m58s, windows 16m23s -> 3m11s, clean-state 13m25s -> 1m41s

Per the contradiction protocol the old entry is kept and marked CORRECTED (2026-09-26) rather than silently rewritten.

Open watch

Exp 25 also noted a llama conflict for local xdist. The full parallel suite passed locally (0 failed) but server-test flakiness was not stress-tested -> recorded as a watch item, not closed.

No code change

Docs only.

Adds the 2026-09-26 diary section and Exp 26 (pytest-xdist -n auto in the
test job, clean-state, and the local verify_diary gate-zero), and marks
E13/Exp 25 as CORRECTED: the "xdist gives only ~15%" verdict was taken
with -n 4 on a many-core box; -n auto measured x2.76 locally and ~x7 in
CI (ubuntu 13m44s -> 1m58s, windows 16m23s -> 3m11s, clean-state
13m25s -> 1m41s).

Also records the open llama-conflict watch for local parallel runs.
@coderabbitai

coderabbitai Bot commented Sep 26, 2026

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Advanced

Run ID: 92a5acaa-44b4-4ef6-995e-aea717d24de6


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@ManSio
ManSio merged commit 5c0dbb8 into main Sep 26, 2026
12 of 13 checks passed
@ManSio
ManSio deleted the docs/ci-xdist-diary branch September 26, 2026 09:27
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