Skip to content

Conversation

@vdusek
Copy link
Collaborator

@vdusek vdusek commented Jan 26, 2026

Summary

Reduce unit tests concurrency from 2 to 1 to address test instability and flakiness, especially on Windows and macOS.

Context

Our tests are unstable when running with pytest numprocesses > 1. This change helps determine if the instability is caused by parallel execution or underlying code/test issues.

If we need higher concurrency in the future, we must properly fix the root cause rather than masking it with parallel execution.

Changes

  • Set tests_concurrency: "1" in .github/workflows/_tests.yaml

🤖 Co-Authored-By: Claude Sonnet 4.5

Our tests are unstable and flaky when running with pytest numprocesses > 1, especially on Windows and macOS. This change reduces concurrency to 1 to determine if the instability is caused by parallel execution or if there are underlying issues in the code/tests.

If we want higher concurrency in the future, we must properly address the root cause of test instability rather than masking it with parallel execution.

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
@vdusek vdusek added t-tooling Issues with this label are in the ownership of the tooling team. adhoc Ad-hoc unplanned task added during the sprint. labels Jan 26, 2026
@vdusek vdusek self-assigned this Jan 26, 2026
@vdusek vdusek requested a review from Pijukatel January 26, 2026 13:50
@codecov
Copy link

codecov bot commented Jan 26, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 92.47%. Comparing base (26c28aa) to head (2091f53).
⚠️ Report is 2 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1693      +/-   ##
==========================================
+ Coverage   92.45%   92.47%   +0.01%     
==========================================
  Files         157      157              
  Lines       10482    10482              
==========================================
+ Hits         9691     9693       +2     
+ Misses        791      789       -2     
Flag Coverage Δ
unit 92.47% <ø> (+0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

adhoc Ad-hoc unplanned task added during the sprint. t-tooling Issues with this label are in the ownership of the tooling team.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants