-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
Unmute remote store test testPressureServiceStats #8458
Conversation
Signed-off-by: Kunal Kotwani <kkotwani@amazon.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @kotwanikunal for the effort on this issue.
In order to evaluate current state on flakyness, I would suggest to run this test locally without seed for 100+ iterations.
|
Gradle Check (Jenkins) Run Completed with:
|
Codecov Report
@@ Coverage Diff @@
## main #8458 +/- ##
============================================
- Coverage 70.89% 70.85% -0.05%
- Complexity 56927 56964 +37
============================================
Files 4758 4758
Lines 269219 269219
Branches 39407 39407
============================================
- Hits 190876 190760 -116
- Misses 62268 62389 +121
+ Partials 16075 16070 -5 |
…#8458) Signed-off-by: Kunal Kotwani <kkotwani@amazon.com>
…#8458) Signed-off-by: Kunal Kotwani <kkotwani@amazon.com>
…#8458) Signed-off-by: Kunal Kotwani <kkotwani@amazon.com> Signed-off-by: sahil buddharaju <sahilbud@amazon.com>
…#8458) Signed-off-by: Kunal Kotwani <kkotwani@amazon.com>
…#8458) Signed-off-by: Kunal Kotwani <kkotwani@amazon.com> Signed-off-by: Shivansh Arora <hishiv@amazon.com>
Description
SegmentReplicationUsingRemoteStoreIT.testPressureServiceStats()
as it does not fail with the linked seedsRelated Issues
Resolves #7592
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.