Skip to content

Conversation

fantix
Copy link
Contributor

@fantix fantix commented May 28, 2020

After the first connect attempt before the listener is ready, the socket needs time to "recover" to make the next connect call. On Linux, a second retry will do. On Windows, the waiting time is unpredictable; and on FreeBSD the socket may never come back because it's a loopback address. Here we'll just retry for a few times, and have to skip the test if it's not working. See also:

https://bugs.python.org/issue30064

@bedevere-bot bedevere-bot added tests Tests in the Lib/test dir awaiting review labels May 28, 2020
@fantix fantix marked this pull request as ready for review May 28, 2020 17:48
@fantix fantix requested review from 1st1 and asvetlov as code owners May 28, 2020 17:48
@1st1 1st1 added the needs backport to 3.9 only security fixes label May 28, 2020
@1st1 1st1 merged commit dc4eee9 into python:master May 28, 2020
@miss-islington
Copy link
Contributor

Thanks @fantix for the PR, and @1st1 for merging it 🌮🎉.. I'm working now to backport this PR to: 3.9.
🐍🍒⛏🤖

@1st1
Copy link
Member

1st1 commented May 28, 2020

Thanks, Fantix!

@bedevere-bot
Copy link

GH-20503 is a backport of this pull request to the 3.9 branch.

@bedevere-bot bedevere-bot removed the needs backport to 3.9 only security fixes label May 28, 2020
miss-islington added a commit that referenced this pull request May 28, 2020
…-20494) (cherry picked from commit dc4eee9) Co-authored-by: Fantix King <fantix.king@gmail.com>
@fantix
Copy link
Contributor Author

fantix commented May 28, 2020

Thanks!

@fantix fantix deleted the bpo-30064/skip-unstable-test branch May 28, 2020 23:02
CuriousLearner added a commit to CuriousLearner/cpython that referenced this pull request May 30, 2020
* 'master' of github.com:python/cpython: (497 commits) bpo-40061: Fix a possible refleak in _asynciomodule.c (pythonGH-19748) bpo-40798: Generate a different message for already removed elements (pythonGH-20483) closes bpo-29017: Update the bindings for Qt information with PySide2 (pythonGH-20149) bpo-39885: Make IDLE context menu cut and copy work again (pythonGH-18951) bpo-29882: Add an efficient popcount method for integers (python#771) Further de-linting of zoneinfo module (python#20499) bpo-40780: Fix failure of _Py_dg_dtoa to remove trailing zeros (pythonGH-20435) Indicate that abs() method accept argument that implement __abs__(), just like call() method in the docs (pythonGH-20509) bpo-39040: Fix parsing of email mime headers with whitespace between encoded-words. (pythongh-17620) bpo-40784: Fix sqlite3 deterministic test (pythonGH-20448) bpo-30064: Properly skip unstable loop.sock_connect() racing test (pythonGH-20494) Note the output ordering of combinatoric functions (pythonGH-19732) bpo-40474: Updated coverage.yml to better report coverage stats (python#19851) bpo-40806: Clarify that itertools.product immediately consumes its inpt (pythonGH-20492) bpo-1294959: Try to clarify the meaning of platlibdir (pythonGH-20332) bpo-37878: PyThreadState_DeleteCurrent() was not removed (pythonGH-20489) bpo-40777: Initialize PyDateTime_IsoCalendarDateType.tp_base at run-time (pythonGH-20493) bpo-40755: Add missing multiset operations to Counter() (pythonGH-20339) bpo-25920: Remove socket.getaddrinfo() lock on macOS (pythonGH-20177) bpo-40275: Fix test.support.threading_helper (pythonGH-20488) ...
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

6 participants