-
-
Notifications
You must be signed in to change notification settings - Fork 55
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
Increase UnixNoGilBuild timeout #476
Conversation
Example: on build 1985, test_math failed with a timeout (20 min)
The next build is a success, but test_math was the slowest test: test_math bad performance is a known issue, unrelated to Free Threading: python/cpython#110327 |
cc @itamaro |
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.
Thanks Victor!
@@ -169,6 +169,9 @@ class UnixNoGilBuild(UnixBuild): | |||
buildersuffix = ".nogil" | |||
configureFlags = ["--with-pydebug", "--disable-gil"] | |||
factory_tags = ["nogil"] | |||
# 2024-04-11: Free-threading can still be slower than regular build in some | |||
# code paths, so tolerate longer timeout. | |||
test_timeout = int(TEST_TIMEOUT * 1.5) | |||
|
|||
|
|||
class UnixNoGilRefleakBuild(UnixBuild): |
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.
Should we increase the timeout for this one too?
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.
UnixNoGilRefleakBuild already uses TEST_TIMEOUT * 10
and I didn't see any test failing with this timeout. I prefer to not make it bigger (until a test fails with timeout), the timeout is already larger than 3 hours! (worker timeout: 3 hour 25 min
)
Merged, thanks for the review. |
No description provided.