-
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 test - handle IllegalStateException and throw AssertionError to use assertbusy #11981
fix flaky test - handle IllegalStateException and throw AssertionError to use assertbusy #11981
Conversation
…r to use assertbusy Signed-off-by: bansvaru <[email protected]>
Compatibility status:Checks if related components are compatible with change 4a631f2 Incompatible componentsIncompatible components: [https://github.com/opensearch-project/custom-codecs.git, https://github.com/opensearch-project/performance-analyzer.git, https://github.com/opensearch-project/cross-cluster-replication.git, https://github.com/opensearch-project/performance-analyzer-rca.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/notifications.git, https://github.com/opensearch-project/opensearch-oci-object-storage.git, https://github.com/opensearch-project/job-scheduler.git, https://github.com/opensearch-project/neural-search.git, https://github.com/opensearch-project/common-utils.git, https://github.com/opensearch-project/geospatial.git, https://github.com/opensearch-project/security.git, https://github.com/opensearch-project/anomaly-detection.git, https://github.com/opensearch-project/index-management.git, https://github.com/opensearch-project/ml-commons.git, https://github.com/opensearch-project/alerting.git, https://github.com/opensearch-project/sql.git, https://github.com/opensearch-project/k-nn.git] |
❌ Gradle check result for b191308: 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? |
changes are unrelated to the failing test |
LGTM. How many times have we run the test to test for flakiness fixed? Looks like a build is failing. |
Signed-off-by: bansvaru <[email protected]>
Not able to repro after fix in 5k iterations. Retried the build |
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## main #11981 +/- ##
============================================
+ Coverage 71.41% 71.46% +0.05%
- Complexity 59397 59429 +32
============================================
Files 4923 4923
Lines 279212 279214 +2
Branches 40595 40596 +1
============================================
+ Hits 199408 199552 +144
+ Misses 63223 63040 -183
- Partials 16581 16622 +41 ☔ View full report in Codecov by Sentry. |
…r to use assertbusy (#11981) * fix flaky test - handle IllegalStateException and throw AssertionError to use assertbusy Signed-off-by: bansvaru <[email protected]> * Empty Commit to retry the build Signed-off-by: bansvaru <[email protected]> --------- Signed-off-by: bansvaru <[email protected]> (cherry picked from commit bf5e628) Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
…r to use assertbusy (#11981) (#12109) * fix flaky test - handle IllegalStateException and throw AssertionError to use assertbusy * Empty Commit to retry the build --------- (cherry picked from commit bf5e628) Signed-off-by: bansvaru <[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>
…r to use assertbusy (#11981) (#12109) * fix flaky test - handle IllegalStateException and throw AssertionError to use assertbusy * Empty Commit to retry the build --------- (cherry picked from commit bf5e628) Signed-off-by: bansvaru <[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> (cherry picked from commit 2011c13) Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
…r to use assertbusy (#11981) (#12109) (#12282) * fix flaky test - handle IllegalStateException and throw AssertionError to use assertbusy * Empty Commit to retry the build --------- (cherry picked from commit bf5e628) (cherry picked from commit 2011c13) Signed-off-by: bansvaru <[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>
…r to use assertbusy (opensearch-project#11981) * fix flaky test - handle IllegalStateException and throw AssertionError to use assertbusy Signed-off-by: bansvaru <[email protected]> * Empty Commit to retry the build Signed-off-by: bansvaru <[email protected]> --------- Signed-off-by: bansvaru <[email protected]>
…r to use assertbusy (opensearch-project#11981) * fix flaky test - handle IllegalStateException and throw AssertionError to use assertbusy Signed-off-by: bansvaru <[email protected]> * Empty Commit to retry the build Signed-off-by: bansvaru <[email protected]> --------- Signed-off-by: bansvaru <[email protected]>
…r to use assertbusy (opensearch-project#11981) * fix flaky test - handle IllegalStateException and throw AssertionError to use assertbusy Signed-off-by: bansvaru <[email protected]> * Empty Commit to retry the build Signed-off-by: bansvaru <[email protected]> --------- Signed-off-by: bansvaru <[email protected]> Signed-off-by: Shivansh Arora <[email protected]>
Description
Related Issues
Resolves #10749
Check List
New functionality has been documented.New functionality has javadoc addedFailing checks are inspected and point to the corresponding known issue(s) (See: Troubleshooting Failing Builds)Commit 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.