-
-
Notifications
You must be signed in to change notification settings - Fork 4.9k
Refactor integration and smoke tests CI #9855
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
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #9855 +/- ##
=======================================
Coverage 78.63% 78.63%
=======================================
Files 153 153
Lines 19198 19198
Branches 2546 2546
=======================================
Hits 15096 15096
- Misses 3807 3809 +2
+ Partials 295 293 -2
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
e98f537 to
2c7c397
Compare
auvipy
left a comment
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.
wah only 3 failures! I guess we can now focus on the remaining pr
auvipy
left a comment
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.
only one test is failing now
Yeah but I had to rerun many times. I'll keep investigating bro, LMK if you have any idea/tip! |
|
could be redis multi threading or race condition issue or this could be in billiard or may be not. #9837 (comment) |
…s from 2 -> 3 (fail/retry faster)
a060103 to
6926cec
Compare
|
only one old known test is failing ::test_chain::test_chain_error_handler_with_eta PASSED |
Almost there! |
5165dd0 to
05b10e9
Compare
…rker shutdown" This reverts commit cebcb2b.
…dition (celery#9770)" This reverts commit b10a55c.
This reverts commit ade8cbd.
No description provided.