-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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 SimpleQueryStringIT Tests #12575
Conversation
Compatibility status:Checks if related components are compatible with change bb9d3d6 Incompatible componentsSkipped componentsCompatible componentsCompatible components: [https://github.com/opensearch-project/custom-codecs.git, https://github.com/opensearch-project/geospatial.git, https://github.com/opensearch-project/flow-framework.git, https://github.com/opensearch-project/observability.git, https://github.com/opensearch-project/cross-cluster-replication.git, https://github.com/opensearch-project/job-scheduler.git, https://github.com/opensearch-project/opensearch-oci-object-storage.git, https://github.com/opensearch-project/neural-search.git, https://github.com/opensearch-project/ml-commons.git, https://github.com/opensearch-project/k-nn.git, https://github.com/opensearch-project/security-analytics.git, https://github.com/opensearch-project/asynchronous-search.git, https://github.com/opensearch-project/sql.git, https://github.com/opensearch-project/reporting.git, https://github.com/opensearch-project/notifications.git, https://github.com/opensearch-project/common-utils.git, https://github.com/opensearch-project/anomaly-detection.git, https://github.com/opensearch-project/index-management.git, https://github.com/opensearch-project/alerting.git, https://github.com/opensearch-project/performance-analyzer-rca.git, https://github.com/opensearch-project/security.git, https://github.com/opensearch-project/performance-analyzer.git] |
❕ Gradle check result for 02679f8: 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 #12575 +/- ##
============================================
- Coverage 71.43% 71.39% -0.05%
- Complexity 59960 59963 +3
============================================
Files 4984 4984
Lines 282247 282247
Branches 40952 40952
============================================
- Hits 201617 201504 -113
- Misses 63954 64057 +103
- Partials 16676 16686 +10 ☔ View full report in Codecov by Sentry. |
@kkmr please see it again, it's still blocked. |
@kkewwei Can you please describe how the new values solve flakiness? |
@kotwanikunal simpleQueryStringQuery("foo Bar 19 127.0.0.1")
when CLUSTER_MAX_CLAUSE_COUNT=80, it work will. |
@kkewwei could you please add the comment to the test (where random value is assigned) to explain lower bound selection? thank you |
Signed-off-by: kkewwei <kkewwei@163.com>
Signed-off-by: kkewwei <kkewwei@163.com>
❌ Gradle check result for c32ca4a: 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? |
❌ Gradle check result for 6d5e663: 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? |
❌ Gradle check result for bb9d3d6: 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? |
@reta @kotwanikunal TBR, thank you. |
|
* Fix Flaky SimpleQueryStringIT Tests Signed-off-by: kkewwei <kkewwei@163.com> * add the comment to unit test Signed-off-by: kkewwei <kkewwei@163.com> --------- Signed-off-by: kkewwei <kkewwei@163.com> (cherry picked from commit fd458d6) Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
* Fix Flaky SimpleQueryStringIT Tests * add the comment to unit test --------- (cherry picked from commit fd458d6) Signed-off-by: kkewwei <kkewwei@163.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>
* Fix Flaky SimpleQueryStringIT Tests Signed-off-by: kkewwei <kkewwei@163.com> * add the comment to unit test Signed-off-by: kkewwei <kkewwei@163.com> --------- Signed-off-by: kkewwei <kkewwei@163.com> Signed-off-by: Shivansh Arora <hishiv@amazon.com>
Signed-off-by: kkewwei kkewwei@163.com
Description
org.opensearch.search.query.SimpleQueryStringIT.testDocWithAllTypes is flaky.
Related Issues
Resolves #12574
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.