Skip to content
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

buildbot: support.transient_internet() doesn't catch DNS socket.gaierror #53081

Closed
vstinner opened this issue May 27, 2010 · 6 comments
Closed
Assignees
Labels
tests Tests in the Lib/test dir

Comments

@vstinner
Copy link
Member

BPO 8835
Nosy @pitrou, @vstinner
Files
  • transilient_internet_dns.patch
  • Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.

    Show more details

    GitHub fields:

    assignee = 'https://github.com/vstinner'
    closed_at = <Date 2010-09-10.22:08:31.065>
    created_at = <Date 2010-05-27.21:47:35.558>
    labels = ['tests']
    title = "buildbot: support.transient_internet() doesn't catch DNS socket.gaierror"
    updated_at = <Date 2010-09-10.22:08:31.064>
    user = 'https://github.com/vstinner'

    bugs.python.org fields:

    activity = <Date 2010-09-10.22:08:31.064>
    actor = 'pitrou'
    assignee = 'vstinner'
    closed = True
    closed_date = <Date 2010-09-10.22:08:31.065>
    closer = 'pitrou'
    components = ['Tests']
    creation = <Date 2010-05-27.21:47:35.558>
    creator = 'vstinner'
    dependencies = []
    files = ['17477']
    hgrepos = []
    issue_num = 8835
    keywords = ['patch']
    message_count = 6.0
    messages = ['106617', '106619', '106620', '106622', '106646', '106682']
    nosy_count = 2.0
    nosy_names = ['pitrou', 'vstinner']
    pr_nums = []
    priority = 'normal'
    resolution = 'out of date'
    stage = None
    status = 'closed'
    superseder = None
    type = None
    url = 'https://bugs.python.org/issue8835'
    versions = ['Python 2.7', 'Python 3.2']

    @vstinner
    Copy link
    Member Author

    The nameserver responsible of sha2.hboeck.de is down and this hostname is used in test_ssl. The result is that all trunk buildbots are red (error).

    support.transient_internet() should catch socket.gaierror.

    --

    One example: http://www.python.org/dev/buildbot/all/builders/amd64%20gentoo%20trunk/builds/1007

    Traceback (most recent call last):
      File ".../Lib/test/test_ssl.py", line 261, in test_algorithms
        s.connect(remote)
      ...
    gaierror: [Errno -2] Name or service not known

    @vstinner vstinner added the tests Tests in the Lib/test dir label May 27, 2010
    @vstinner
    Copy link
    Member Author

    Patch to catch gaierror(EAI_NODATA) and gaierror(EAI_NONAME).

    @pitrou
    Copy link
    Member

    pitrou commented May 27, 2010

    Looks ok to me.

    @vstinner
    Copy link
    Member Author

    I commited the patch as r81571 in trunk. Wait for the buildbot before porting it to other branches.

    @vstinner
    Copy link
    Member Author

    There is another error:

    test test_ssl failed -- Traceback (most recent call last):
      File "/scratch/pybot-buildarea/trunk.klose-ubuntu-i386/build/Lib/test/test_ssl.py", line 261, in test_algorithms
        s.connect(remote)
      File "/scratch/pybot-buildarea/trunk.klose-ubuntu-i386/build/Lib/ssl.py", line 292, in connect
        socket.connect(self, addr)
      File "/scratch/pybot-buildarea/trunk.klose-ubuntu-i386/build/Lib/socket.py", line 222, in meth
        return getattr(self._sock,name)(*args)
    error: [Errno 110] Connection timed out

    transient_internet() should also catch socket.error(errno.ETIMEDOUT).

    --

    See also bpo-8455, bpo-8499 and bpo-8574.

    @pitrou
    Copy link
    Member

    pitrou commented May 28, 2010

    I commited the patch as r81571 in trunk.

    Apparently it's ok.

    @pitrou pitrou closed this as completed Sep 10, 2010
    @ezio-melotti ezio-melotti transferred this issue from another repository Apr 10, 2022
    Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
    Labels
    tests Tests in the Lib/test dir
    Projects
    None yet
    Development

    No branches or pull requests

    2 participants