Skip to content

ci(acceptance): experiment: 3 xdist workers - #129389

Closed
sentry-junior[bot] wants to merge 1 commit into
ci/add-xdist-worker-backend-testsfrom
ci/xdist-exp-acceptance-3-workers
Closed

sentry-junior[bot] wants to merge 1 commit into
ci/add-xdist-worker-backend-testsfrom
ci/xdist-exp-acceptance-3-workers

Conversation

@sentry-junior

@sentry-junior sentry-junior Bot commented Oct 30, 2026

Copy link
Copy Markdown
Contributor

Experiment, not for merge. Stacked on #129338.

Sets acceptance XDIST_WORKERS from 2 to 3 on ubuntu-26.04 (amd64), to compare acceptance test time with #129338 (2 workers) and the 4-worker experiment. The results will be posted here, then this PR will be closed.

via @joshuarli.

--

View Session [Sentry]

Co-Authored-By: Joshua Li <14209781+joshuarli@users.noreply.github.com>
@sentry-junior

sentry-junior Bot commented Oct 30, 2026

Copy link
Copy Markdown
Contributor Author

Result: 3 acceptance workers doesn't pay off on x64. Closing; acceptance stays at 2 in #129338.

All runs at 10 shards, green, fitted per shard and per runner CPU type (residual noise ~5% per shard):

Workers Runs Test step Job
2 #129395 (41142270035) baseline baseline
3 this PR (41140659912, 41142236449) −2.3% (noise) +4.8%
4 #129390 (41140876860) +5.3% +14.0%

The 3rd worker barely moves test time (the runner is already CPU-bound with Chrome) and costs an extra per-worker Snuba bootstrap in setup.

@sentry-junior sentry-junior Bot closed this Oct 30, 2026
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