-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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 testClusterStateBatchedUpdates test #10922
Fix flaky testClusterStateBatchedUpdates test #10922
Conversation
Signed-off-by: Aman Khare <amkhar@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.
LGTM.
Gradle Check (Jenkins) Run Completed with:
|
Codecov Report
@@ Coverage Diff @@
## main #10922 +/- ##
============================================
- Coverage 71.22% 71.15% -0.08%
- Complexity 58678 58682 +4
============================================
Files 4869 4872 +3
Lines 276496 276682 +186
Branches 40203 40219 +16
============================================
- Hits 196936 196860 -76
- Misses 63114 63440 +326
+ Partials 16446 16382 -64
|
Compatibility status:Checks if related components are compatible with change afc5919 Incompatible componentsIncompatible components: [https://github.com/opensearch-project/cross-cluster-replication.git, https://github.com/opensearch-project/performance-analyzer.git] Skipped componentsCompatible componentsCompatible components: [https://github.com/opensearch-project/security.git, https://github.com/opensearch-project/alerting.git, https://github.com/opensearch-project/index-management.git, https://github.com/opensearch-project/anomaly-detection.git, https://github.com/opensearch-project/job-scheduler.git, https://github.com/opensearch-project/sql.git, https://github.com/opensearch-project/asynchronous-search.git, https://github.com/opensearch-project/common-utils.git, https://github.com/opensearch-project/observability.git, https://github.com/opensearch-project/k-nn.git, https://github.com/opensearch-project/reporting.git, https://github.com/opensearch-project/security-analytics.git, https://github.com/opensearch-project/custom-codecs.git, https://github.com/opensearch-project/opensearch-oci-object-storage.git, https://github.com/opensearch-project/performance-analyzer-rca.git, https://github.com/opensearch-project/ml-commons.git, https://github.com/opensearch-project/geospatial.git, https://github.com/opensearch-project/notifications.git, https://github.com/opensearch-project/neural-search.git] |
@amkhar can you strike out things from checklist that don't apply for the build checklist to be green. |
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.
Code coverage checks are failing, might need to be addressed
Thanks @Bukhtawar for approval. This PR is just for test change no code changes are part of this PR. |
Going ahead with this as its blocking a lot of checks and gc passes. |
Signed-off-by: Aman Khare <amkhar@amazon.com> Co-authored-by: Aman Khare <amkhar@amazon.com> (cherry picked from commit 44a9f18) Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
(cherry picked from commit 44a9f18) Signed-off-by: Aman Khare <amkhar@amazon.com> Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> Co-authored-by: Aman Khare <amkhar@amazon.com>
Signed-off-by: Aman Khare <amkhar@amazon.com> Co-authored-by: Aman Khare <amkhar@amazon.com> Signed-off-by: Shivansh Arora <hishiv@amazon.com>
Description
Move the assertion to more deterministic test where cluster state update will happen for sure. Executed the test 1000 time locally in a loop to verify it's not going to be flaky anymore.
Related Issues
Resolves #10899
Check List
- [x] New functionality has been documented.- [x] New functionality has javadoc added- [x] Commit changes are listed out in CHANGELOG.md file (See: Changelog)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.