Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix flaky ServerStatusManagerIntegrationTest #10

Closed
wants to merge 6 commits into from
Closed

Conversation

ikhoon
Copy link
Owner

@ikhoon ikhoon commented Apr 23, 2024

No description provided.

Motivation:

This PR tries to solve the flakiness of `ServerStatusManagerIntegrationTest`
in a different way. See line#948

This workaround minimizes port collisions by ensuring that all ports are
available when a Central Dogma cluster starts.

Modifications:

- Check if all ports are open when restarting
  `CentralDogmaReplicationExtension`.

Result:

Less flakiness
@ikhoon ikhoon force-pushed the fix-flaky-test-v2 branch from 83d2f39 to 7d9a211 Compare April 23, 2024 09:02
@codecov-commenter
Copy link

Codecov Report

Attention: Patch coverage is 88.23529% with 4 lines in your changes are missing coverage. Please review.

Project coverage is 70.50%. Comparing base (cfb8f9c) to head (96a680f).
Report is 20 commits behind head on main.

Files Patch % Lines
...ing/internal/CentralDogmaReplicationExtension.java 88.23% 1 Missing and 3 partials ⚠️

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@             Coverage Diff              @@
##               main      #10      +/-   ##
============================================
+ Coverage     66.77%   70.50%   +3.72%     
- Complexity     3514     3680     +166     
============================================
  Files           370      376       +6     
  Lines         14476    14796     +320     
  Branches       1553     1591      +38     
============================================
+ Hits           9667    10432     +765     
+ Misses         3932     3433     -499     
- Partials        877      931      +54     

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

@ikhoon ikhoon closed this Jun 14, 2024
@ikhoon ikhoon deleted the fix-flaky-test-v2 branch August 30, 2024 09:11
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