-
Notifications
You must be signed in to change notification settings - Fork 31
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
Retry flaky tests #449
Retry flaky tests #449
Conversation
Signed-off-by: Owais Kazi <[email protected]>
30bf56f
to
55ba20a
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #449 +/- ##
=========================================
Coverage 71.88% 71.88%
Complexity 620 620
=========================================
Files 78 78
Lines 3126 3126
Branches 236 236
=========================================
Hits 2247 2247
Misses 772 772
Partials 107 107 ☔ View full report in Codecov by Sentry. |
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.
Cool app.
We do not need to apply it to all tests. Just IT. And let’s limit to one retry.
a58a156
to
6bb345c
Compare
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.
Just a small comment, but looks got to me. I'm still not sure why we're encountering these multi-node failures on JDK 21.0.2 but not JDK 21.0.1
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.
Just a small comment, but looks got to me. I'm still not sure why we're encountering these multi-node failures on JDK 21.0.2 but not JDK 21.0.1
looks good to me, but I agree with Josh that we need another issue to solve these failures, the multi node flakiness is still a little concerning |
Signed-off-by: Owais Kazi <[email protected]>
6bb345c
to
5843aec
Compare
* Retry flaky tests Signed-off-by: Owais Kazi <[email protected]> * Addressed PR Comments Signed-off-by: Owais Kazi <[email protected]> --------- Signed-off-by: Owais Kazi <[email protected]> (cherry picked from commit 03cdb99) Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Retry flaky tests (#449) * Retry flaky tests * Addressed PR Comments --------- (cherry picked from commit 03cdb99) Signed-off-by: Owais Kazi <[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>
Description
Retry flaky tests using
org.gradle.test-retry
plugin.Issues Resolved
Closes #442
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.