Skip to content

Conversation

@mawasile
Copy link
Contributor

This pull request makes a small adjustment to the test workflow configuration. The change reduces the parallelism of Go tests from 3 to 1, which can help with test reliability in environments where tests may interfere with each other.

  • Reduced the Go test parallelism from 3 to 1 in the .github/workflows/run_tests.yml workflow to improve test reliability.

Copilot AI review requested due to automatic review settings December 18, 2025 09:15
@mawasile mawasile requested a review from a team as a code owner December 18, 2025 09:15
@mawasile mawasile added run-acceptance-tests PR tag for triggering acceptance tests skip-changelog labels Dec 18, 2025
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This pull request addresses test reliability issues by reducing the parallelism of Go tests in the CI/CD workflow from 3 to 1. While this may increase test execution time, it can help prevent race conditions and test interference in the acceptance test environment.

Key Changes

  • Reduced Go test parallelism from -p 3 to -p 1 in the test workflow to improve reliability

@mawasile mawasile self-assigned this Dec 18, 2025
@codecov-commenter
Copy link

codecov-commenter commented Dec 18, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 70.03%. Comparing base (f8775b6) to head (e0d2dcd).
⚠️ Report is 5 commits behind head on main.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1018      +/-   ##
==========================================
+ Coverage   68.67%   70.03%   +1.36%     
==========================================
  Files         131      131              
  Lines       16191    16190       -1     
==========================================
+ Hits        11119    11339     +220     
+ Misses       4029     3756     -273     
- Partials     1043     1095      +52     

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

@mawasile mawasile changed the title Mawasile/fix test timeouts Fix acceptance test timeouts Dec 19, 2025
@mawasile mawasile removed the run-acceptance-tests PR tag for triggering acceptance tests label Dec 22, 2025
Copy link
Contributor

@polatengin polatengin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@polatengin polatengin merged commit 344d009 into main Dec 22, 2025
15 checks passed
@polatengin polatengin deleted the mawasile/fix-test-timeouts branch December 22, 2025 23:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants