-
Notifications
You must be signed in to change notification settings - Fork 104
ci: mark test_parallelize as flaky #1105
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
ci: mark test_parallelize as flaky #1105
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #1105 +/- ##
=======================================
Coverage 66.21% 66.21%
=======================================
Files 44 44
Lines 7163 7163
Branches 1217 1217
=======================================
Hits 4743 4743
Misses 1943 1943
Partials 477 477 🚀 New features to boost your workflow:
|
|
No, that makes no sense, adding 100 integers together should timeout after 30ms not 30s. |
|
I am not sure, it was always problematic. I recently noticed also fails locally when backend is not loky which is skipped in CI. Is it ok if I have an issue about this and we just merge for now? so that the CI's arent flaky for now |
|
If restarting helps, you can try |
|
Like this is clearly broken, more timeout won’t help. |
|
ok it passes, thank you ! |
|
@flying-sheep I created an issue here #1111 could we just merge it for now? |
fixes: #1104