Skip to content

Reduce CI test parallelism cap from 8 to 4#714

Merged
rhysparry merged 1 commit into
mainfrom
rhys/eft-3214/reduce-test-parallelism-cap
May 7, 2026
Merged

Reduce CI test parallelism cap from 8 to 4#714
rhysparry merged 1 commit into
mainfrom
rhys/eft-3214/reduce-test-parallelism-cap

Conversation

@rhysparry
Copy link
Copy Markdown
Contributor

@rhysparry rhysparry commented May 7, 2026

Summary

  • Lowers LevelOfParallelismInTeamCity() cap from 8 to 4 in CustomLevelOfParallelismAttribute. On our 8 vCPU Standard_D8as_v4 agent this drops in-flight tests from 6 to 4.
  • Updates the doc-comment table in the same file to reflect the new mapping.
  • Goal: reduce contention-driven flakiness on CI agents — particularly the "connection forcibly closed" / TcpClientManager dispose races and Stopwatch-vs-wall-clock drift seen on Windows 2012R2 under load. Filed in isolation (off main, not stacked on EFT-3214's other branch) so the effect on the next CI run is observable on its own.

Test plan

  • CI runs to completion on Windows 2012R2 net8.0 agent
  • Compare failed-test count vs recent main-nightly and the previous PR Fix race condition in ObserveUnauthorizedPollingConnections test #713 build
  • Spot-check a couple of timing-sensitive tests (HalibutTimeoutsAndLimits_AppliesToTcpClientReceiveTimeout, WhenThenNetworkIsPaused_*) for whether Stopwatch.Elapsed assertions now hold
  • Confirm overall test wall-clock isn't substantially worse (we expect some increase from less parallelism)

🤖 Generated with Claude Code

Lower contention on CI agents to reduce timing-coupled and connection
race flakiness. The test suite spawns out-of-process compatibility
binaries and many short-lived loopback TCP connections; at 6+ in-flight
tests on an 8 vCPU agent this regularly produces ephemeral-port and
TcpClientManager dispose races that show up as "connection forcibly
closed" failures, plus Stopwatch-vs-wall-clock drift on Windows 2012R2
guests under load.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@rhysparry rhysparry requested a review from a team as a code owner May 7, 2026 03:40
@rhysparry
Copy link
Copy Markdown
Contributor Author

Test run completed first run in ~28m.

@rhysparry rhysparry merged commit 10ea710 into main May 7, 2026
17 checks passed
@rhysparry rhysparry deleted the rhys/eft-3214/reduce-test-parallelism-cap branch May 7, 2026 06:01
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.

2 participants