-
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
Fixing the tests for concurrent search #11207
Conversation
Compatibility status:Checks if related components are compatible with change 84688b1 Incompatible componentsIncompatible components: [https://github.com/opensearch-project/performance-analyzer.git] Skipped componentsCompatible componentsCompatible components: [https://github.com/opensearch-project/asynchronous-search.git, https://github.com/opensearch-project/security-analytics.git, https://github.com/opensearch-project/observability.git, https://github.com/opensearch-project/reporting.git, https://github.com/opensearch-project/job-scheduler.git, https://github.com/opensearch-project/opensearch-oci-object-storage.git, https://github.com/opensearch-project/sql.git, https://github.com/opensearch-project/common-utils.git, https://github.com/opensearch-project/custom-codecs.git, https://github.com/opensearch-project/ml-commons.git, https://github.com/opensearch-project/security.git, https://github.com/opensearch-project/notifications.git, https://github.com/opensearch-project/anomaly-detection.git, https://github.com/opensearch-project/index-management.git, https://github.com/opensearch-project/neural-search.git, https://github.com/opensearch-project/alerting.git, https://github.com/opensearch-project/cross-cluster-replication.git, https://github.com/opensearch-project/geospatial.git, https://github.com/opensearch-project/k-nn.git, https://github.com/opensearch-project/performance-analyzer-rca.git] |
❌ Gradle check result for f1016d1: FAILURE Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change? |
f1016d1
to
11b883c
Compare
❌ Gradle check result for 11b883c: FAILURE Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change? |
11b883c
to
a99290c
Compare
❌ Gradle check result for a99290c: FAILURE Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change? |
Signed-off-by: Bansi Kasundra <[email protected]>
a99290c
to
84688b1
Compare
❕ Gradle check result for 84688b1: UNSTABLE
Please review all flaky tests that succeeded after retry and create an issue if one does not already exist to track the flaky failure. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #11207 +/- ##
============================================
+ Coverage 71.15% 71.25% +0.09%
- Complexity 58783 58815 +32
============================================
Files 4885 4885
Lines 277199 277199
Branches 40285 40285
============================================
+ Hits 197247 197517 +270
+ Misses 63448 63142 -306
- Partials 16504 16540 +36 ☔ View full report in Codecov by Sentry. |
Signed-off-by: Bansi Kasundra <[email protected]> (cherry picked from commit 12c3d07) Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
(cherry picked from commit 12c3d07) Signed-off-by: Bansi Kasundra <[email protected]> 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>
Signed-off-by: Bansi Kasundra <[email protected]>
Signed-off-by: Bansi Kasundra <[email protected]>
Signed-off-by: Bansi Kasundra <[email protected]> Signed-off-by: Shivansh Arora <[email protected]>
Description
Follow-up to #11076.
Added indexRandomForConcurrentSearch method to the tests that were failing earlier for concurrent search case.
Related Issues
Resolves #11074
Check List
New functionality has been documented.New functionality has javadoc addedCommit changes are listed out in CHANGELOG.md file (See: Changelog)Public documentation issue/PR createdBy 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.