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 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.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants