Skip to content

test_sendfile_fallback_close_peer_in_the_middle_of_receiving false failure on macOS #98174

Description

@fancidev

The test_sendfile_fallback_close_peer_in_the_middle_of_receiving test in test.test_asyncio.test_sendfile.SelectEventLoopTests fails with error “OSError: [Errno 41] Protocol wrong type for socket” intermittently on macOS.

I have seen this error multiple times across Python 3.8-3.11rc1 CI test runs on GitHub Actions. For a recent example, see https://github.com/fancidev/qtinter/actions/runs/3219024415/jobs/5263887528 (this is a custom build run, but the test steps involves no custom code)

This is in fact nothing new — a similar issue was reported and fixed in #88395

Maybe a similar fix to the test case would suit.

Activity

  1. fancidev commented on Oct 11, 2022

    @fancidev
    ContributorAuthor

    Another CI test run with the same error (Python 3.7):
    Python-3.7-PySide2-macos-latest

  2. sobolevn commented on Oct 12, 2022

    @sobolevn
    Member

    Can you please try the latest version? Will it still fail for you?
    Refs #97546

  3. gvanrossum commented on Oct 12, 2022

    @gvanrossum
    Member

    If it does still fail in 3.11 or main, could you suggest a PR? We're short of experts in this area for sure! And I think our main SSL expert works primarily (only?) on Linux.

  4. akulakov commented on Oct 14, 2022

    @akulakov
    Contributor

    I could not reproduce locally on MacOS 11.5.2 M1, I did a few hundred test runs on both Py 3.9 and latest dev version.

  5. fancidev commented on Oct 16, 2022

    @fancidev
    ContributorAuthor

    @sobolevn: The issue still exists in recent code -- here is a CI run error example on 3.11.0-rc1: https://github.com/fancidev/qtinter/actions/runs/3219024415/jobs/5263887528#logs

    @gvanrossum: I proposed a PR just now. Thanks for reviewing!

    @akulakov: Neither can I reproduce the issue on my dev machine, though I've seen it a few times on CI runs. I suppose the load on CI server makes the issue easier to occur.

  6. added a commit that references this issue on Oct 17, 2022
  7. added 3 commits that reference this issue on Oct 17, 2022
  8. added a commit that references this issue on Oct 17, 2022
  9. added a commit that references this issue on Oct 18, 2022
  10. Repository owner moved this from Todo to Done in asyncioon Oct 21, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    Projects

    • Status
      Done

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions