-
Notifications
You must be signed in to change notification settings - Fork 2.1k
Mute WarmIndexSegmentReplicationIT #18176
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
Mute WarmIndexSegmentReplicationIT #18176
Conversation
This is a new test that is pretty flaky. Muting it now while the feature is still under development. Signed-off-by: Andrew Ross <andrross@amazon.com>
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #18176 +/- ##
============================================
- Coverage 72.52% 72.45% -0.07%
- Complexity 67177 67199 +22
============================================
Files 5476 5476
Lines 310444 310444
Branches 45120 45120
============================================
- Hits 225144 224929 -215
- Misses 66870 67199 +329
+ Partials 18430 18316 -114 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
@skumawat2025 Take a look at the impacted PRs in #18157. There are older failures, such as this one for testIndexReopenClose from March 19. I've also seen failures for |
@skumawat2025 You can also look at the stats from the Gradle Check Dashboard. It looks like the first failure was on March 11 and 10 different tests have failed. |
Okay, I see the issue was created 2 days ago, which is why the failure went unnoticed.. @andrross We can mute these for now. Will work on fixing these soon. |
Thanks @skumawat2025!
The automation doesn't open issues until the test has failed a couple times on the post-commit triggered runs to filter out failures that were caused by unmerged code in PRs.
Yeah, the primary reason I'm on a crusade now to stabilize the tests is so that new flakiness doesn't get lost in all the noise that we currently have. |
This is a new test that is pretty flaky. Muting it now while the feature is still under development.
@gbbafna @skumawat2025 Please take a look. If there's a quick fix we can discard this PR, but otherwise I'd prefer to merge this in the meantime to stabilize the tests.
Related to #18157
Check List
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.