X Tutup
Skip to content

[3.13] gh-145548: Don't use VMADDR_CID_LOCAL from socket#145735

Open
encukou wants to merge 1 commit intopython:3.13from
encukou:3.13-VMADDR_CID_LOCAL
Open

[3.13] gh-145548: Don't use VMADDR_CID_LOCAL from socket#145735
encukou wants to merge 1 commit intopython:3.13from
encukou:3.13-VMADDR_CID_LOCAL

Conversation

@encukou
Copy link
Member

@encukou encukou commented Mar 10, 2026

VMADDR_CID_LOCAL was added to socket in 3.14.
For 3.13, the test needs a local constant in setUp(), as in clientSetUp().

VMADDR_CID_LOCAL was added to `socekt` in 3.14. The test needs a
local constant in setUp(), as in clientSetUp().
@encukou
Copy link
Member Author

encukou commented Mar 10, 2026

!buildbot Fedora.Stable$

@bedevere-bot
Copy link

The regex 'Fedora.Stable$' did not match any buildbot builder. Is the requested builder in the list of stable builders?

@encukou
Copy link
Member Author

encukou commented Mar 10, 2026

!buildbot Fedora.Stable

@bedevere-bot
Copy link

🤖 New build scheduled with the buildbot fleet by @encukou for commit 9a6de87 🤖

Results will be shown at:

https://buildbot.python.org/all/#/grid?branch=refs%2Fpull%2F145735%2Fmerge

The command will test the builders whose names match following regular expression: Fedora.Stable

The builders matched are:

  • AMD64 Fedora Stable Refleaks PR
  • AMD64 Fedora Stable PR
  • PPC64LE Fedora Stable Refleaks PR
  • aarch64 Fedora Stable LTO + PGO PR
  • PPC64LE Fedora Stable LTO PR
  • aarch64 Fedora Stable Refleaks PR
  • aarch64 Fedora Stable Clang PR
  • PPC64LE Fedora Stable Clang PR
  • s390x Fedora Stable Clang Installed PR
  • AMD64 Fedora Stable Clang PR
  • PPC64LE Fedora Stable Clang Installed PR
  • PPC64LE Fedora Stable PR
  • AMD64 Fedora Stable Clang Installed PR
  • s390x Fedora Stable LTO PR
  • s390x Fedora Stable PR
  • s390x Fedora Stable Clang PR
  • s390x Fedora Stable Refleaks PR
  • PPC64LE Fedora Stable LTO + PGO PR
  • AMD64 Fedora Stable LTO + PGO PR
  • aarch64 Fedora Stable LTO PR
  • aarch64 Fedora Stable PR
  • aarch64 Fedora Stable Clang Installed PR
  • s390x Fedora Stable LTO + PGO PR
  • AMD64 Fedora Stable LTO PR

Copy link
Member

@vstinner vstinner left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

I tested manually on Fedora Stable x86-64: the change works as expected. The test is skipped:

testStream (test.test_socket.ThreadedVSOCKSocketStreamTest.testStream) ... skipped "bind() failed with OSError(99, 'Cannot assign requested address')"

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

awaiting merge skip news tests Tests in the Lib/test dir

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

X Tutup