Skip to content

bpo-37553: SendfileUsingSendTest tests timeout too short for Windows … #14716

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

Merged
merged 1 commit into from
Jul 14, 2019

Conversation

paulmon
Copy link
Contributor

@paulmon paulmon commented Jul 11, 2019

The timeout error reproduces very infrequently when test_socket tests are run manually on a Raspberry Pi 3 running Windows IoT Core. I was able to determine that this timeout is the one causing the failure by making it smaller (0.01 seconds) which caused the tests in question to fail every time.

I would like to increase the timeout only if the test machine is running Windows ARM32 (aka ARM).

@zooba

https://bugs.python.org/issue37553

@mangrisano
Copy link
Contributor

/cc @giampaolo

@zooba
Copy link
Member

zooba commented Jul 14, 2019

I'll merge this, but if it doesn't help the buildbot then let's revert it @paulmon

@zooba zooba merged commit 52c8c09 into python:master Jul 14, 2019
@zooba
Copy link
Member

zooba commented Jul 14, 2019

If it does help, then we can backport to 3.8. But the buildbot is running against 3.x only now.

@paulmon
Copy link
Contributor Author

paulmon commented Jul 15, 2019

The first pass with this fix passed the test_socket tests. We need more test passes to see if it's still randomly failing. I'll keep an eye on it.

@paulmon paulmon deleted the buildbot_socket_timeout branch July 17, 2019 20:29
@paulmon
Copy link
Contributor Author

paulmon commented Jul 19, 2019

It looks like increasing this timeout works. The socket tests have all passed on the Windows ARM32 buildbot workers since making this change.

lisroach pushed a commit to lisroach/cpython that referenced this pull request Sep 10, 2019
DinoV pushed a commit to DinoV/cpython that referenced this pull request Jan 14, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
skip news tests Tests in the Lib/test dir
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants