Skip to content

bpo-30064: Properly skip unstable loop.sock_connect() racing test #20494

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
May 28, 2020

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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants