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

tests.greendns_test.TestGetaddrinfo failing: assert udp in filt_res, res #373

Closed
onovy opened this issue Jan 8, 2017 · 8 comments
Closed

Comments

@onovy
Copy link
Contributor

onovy commented Jan 8, 2017

Hi,

I'm Debian maintainer of python-eventlet package.

I just got this bugreport: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=850508

That means, package can't be build on current Debian unstable. Reason is:

======================================================================
FAIL: test_getaddrinfo (tests.greendns_test.TestGetaddrinfo)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/<<PKGBUILDDIR>>/tests/greendns_test.py", line 470, in test_getaddrinfo
    assert udp in filt_res
AssertionError

======================================================================
FAIL: test_getaddrinfo_idn (tests.greendns_test.TestGetaddrinfo)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/<<PKGBUILDDIR>>/tests/greendns_test.py", line 484, in test_getaddrinfo_idn
    assert udp in filt_res, res
AssertionError: [(2, 1, 6, '', ('127.0.0.2', 22))]

======================================================================
FAIL: test_getaddrinfo_inet (tests.greendns_test.TestGetaddrinfo)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/<<PKGBUILDDIR>>/tests/greendns_test.py", line 494, in test_getaddrinfo_inet
    assert udp in [ai[:3] + (ai[4],) for ai in res]
AssertionError

======================================================================
FAIL: test_getaddrinfo_inet6 (tests.greendns_test.TestGetaddrinfo)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/<<PKGBUILDDIR>>/tests/greendns_test.py", line 504, in test_getaddrinfo_inet6
    assert udp in [ai[:3] + (ai[4],) for ai in res]
AssertionError

----------------------------------------------------------------------
Ran 626 tests in 59.798s

FAILED (SKIP=90, failures=4)

Package is version 0.19.0. I tried to build newest version 0.20.1, and issue is same. Because this package builts fine before, I think it can be related to some dependency upgrade.

Any suggestion?

Thanks.

@temoto
Copy link
Member

temoto commented Jan 8, 2017 via email

@Kobla
Copy link

Kobla commented Jan 10, 2017

Hi,

I tried to build it with following versions:
dnspython 1.14.0-3
dnspython 1.13.0-2
greenlet 0.4.10-1
greenlet 0.4.9-2

But issue is same.

@temoto
Copy link
Member

temoto commented Jan 10, 2017

Is it possible to get a big diff of everything that changed since successful build? AFAIK Debian unstable enjoys rolling updates. Maybe there's some kind of repository which specifies current state of packages and it's possible to rollback in time? I'm not familiar with Debian internal kitchen.

Kobla pushed a commit to Kobla/eventlet that referenced this issue Jan 12, 2017
Asserting udp for ssh introduce test failures under debian unstable
because netbase 5.4 removed ssh (22/udp).
This fix changes ssh -> domain, which better suit for test name and
allows tcp and udp simultaneously.
@Kobla
Copy link

Kobla commented Jan 12, 2017

Hi, problem fixed.
New version of netbase package causes test failures.
See details in commit message.

@onovy
Copy link
Contributor Author

onovy commented Jan 12, 2017

PR #375

@temoto
Copy link
Member

temoto commented Jan 13, 2017

That was merged. Please, try again with latest code.

pip install https://github.com/eventlet/eventlet/archive/master.zip

@temoto temoto closed this as completed Jan 13, 2017
@temoto temoto reopened this Jan 13, 2017
@temoto temoto added this to the v0.21 milestone Jan 17, 2017
@temoto
Copy link
Member

temoto commented Jan 17, 2017

From PR conversation I assume everything is fine. Please reopen if the problem wasn't fixed.

@temoto temoto closed this as completed Jan 17, 2017
@onovy
Copy link
Contributor Author

onovy commented Jan 18, 2017

yep, it's fixed and uploaded to Debian. Thanks!

clrpackages pushed a commit to clearlinux-pkgs/eventlet that referenced this issue Mar 20, 2020
… 0.25.1

AGSPhoenix (1):
      Add locals() call to example backdoor invocation

Aayush Kasurde (2):
      Modified pyopenssl example using evenlet
      contributing.md

Anthony Sottile (1):
      Use tox's `TOXENV` environment variable

Chris Kerr (1):
      dns: reading /etc/hosts raised DeprecationWarning for universal lines on Python 3.4+

Daniel Alvarez (1):
      greendns: don't contact nameservers if one entry is returned from hosts file

David Szotten (2):
      workaround for pathlib on py 3.7
      release notes and version bump for 0.25.1

Feng (1):
      greenio: Fixed OSError: [WinError 10038] Socket operation on nonsocket

Geoffrey Thomas (1):
      patcher: workaround for monotonic "no suitable implementation"

Gevorg Davoian (1):
      green.zmq: socket.{recv,send}_* signatures did not match recent upstream pyzmq

Haikel Guemar (1):
      Drop OpenSSL.rand support

Hugo (1):
      Add Python 3.6

James Page (1):
      Avoid dependency on enum-compat

Jaume Marhuenda (2):
      support.greendns: ImportError when dns.rdtypes was imported before eventlet
      greendns: resolving over TCP produced ValueError

Jesse (1):
      tpool: exception in tpool-ed call leaked memory via backtrace

Julien Kasarherou (1):
      wsgi: make Expect 100-continue field-value case-insensitive.

Junyi (2):
      [bug] reimport submodule as well in patcher.inject (#540)
      Fix compatibility with Python 3.7 ssl.SSLSocket  (#531)

Konstantin Enchant (1):
      websocket: fd leak when client did not close connection properly

Lon Hohberger (4):
      Fix bad ipv6 comparison
      greendns udp: Fix infinite loop when source address mismatch
      tests: Add ipv6 tests for greendns udp() function
      tests: Add ipv4 udp tests for greendns

Marcel Plch (1):
      Fix for Python 3.7 (#506)

Matt Bennett (1):
      greendns: be explicit about expecting bytes from sock.recv

Miguel Grinberg (1):
      socket: context manager support

Miro Hrončok (1):
      Stop using deprecated cgi.parse_qs() to support Python 3.8

Ondřej Kobližek (1):
      Fixed tests.greendns_test.TestGetaddrinfo eventlet/eventlet#373

Ondřej Nový (1):
      Regenerate test crt

Quan Tian (1):
      patcher: set locked RLocks' owner only when patching existing locks

Ralf Haferkamp (1):
      greendns: Treat /etc/hosts entries case-insensitive

Sam Merritt (1):
      pools: put to empty pool would block sometimes

Sergey Shepelev (59):
      zmq: autogenerated documentation was missing
      Explicit environ flag for importing eventlet.__version__ without ignoring import errors
      release: use twine for PyPI upload
      readme: latest dev version was pointing to bitbucket
      tests: patcher_import_patched_defaults was failing in presence of pyopenssl package
      dns: try unqualified queries as top level
      Type check Semaphore, GreenPool arguments; Thanks to Matthew D. Pagel
      test_import_patched_defaults bended to play with pyopenssl>=16.1.0
      v0.20.1 release
      test coverage reports
      tests cleanup, CI with Python 3.6
      python3.6: http.client.request support chunked_encoding
      New timeout error API: .is_timeout=True on exception object
      support: upgrade bundled six to 1.10 (dbfbfc818e3d)
      green.profile: Python3 compatibility; Thanks to Artur Stawiarski
      Timeout was marked deprecated along with TimeoutError by mistake
      tests: socket_resolve_green was giving false fails
      dns: hosts file was consulted after nameservers
      hubs: use monotonic clock by default (bundled package); Thanks to Roman Podoliaka and Victor Stinner
      dns: EVENTLET_NO_GREENDNS option is back, green is still default
      dns: EAI_NODATA was removed from RFC3493 and FreeBSD
      db_pool: proxy Connection.set_isolation_level()
      ssl: RecursionError on Python3.6+; Thanks to justdoit0823@github and Gevent developers
      wsgi: log_output=False was not disabling startup and accepted messages
      v0.21.0 release
      update monotonic 1.3 5c0322dc559bf961f7e111d97cd3ed9ab5c1a73b
      queue: empty except was catching too much
      wsgi: push deprecated options one step
      wsgi: close idle connections (also applies to websockets)
      convenience: skip SO_REUSEPORT for bind on random port (0)
      convenience: SO_REUSEPORT is not available on WSL platform (Linux on Windows)
      green.subprocess: keep CalledProcessError identity; Thanks to Linbing@github
      support: upgrade bundled dnspython to 1.16.0 (22e9de1d7957e)
      convenience: (SO_REUSEPORT) socket.error is not OSError on Python 2; Thanks to JacoFourie@github
      travis: codecov flags format was running `tr` with invalid arguments
      greendns: early socket.timeout was breaking IO retry loops
      init: second workaround for monotonic "no suitable implementation"; Thanks to Geoffrey Thomas
      Travis broke ipv6, allow failure; test against Python 2.7
      travis: crutch to get ipv6 back
      v0.22.0 release
      event: Event.wait() timeout=None argument to be compatible with upstream CPython
      v0.22.1 release
      support: psycopg2_patcher import module, not function
      travis: update test dependencies, use psycopg2-binary
      moved function eventlet.support.capture_stderr to tests
      travis: allow fail python 3.7 see issue eventlet/eventlet#475
      wsgi: latin-1 encoding dance for environ[PATH_INFO]
      green.threading: current_thread() did not see new monkey-patched threads; Thanks to Jake Tesler
      v0.23.0 release
      Drop support for Python2.6 and python-epoll package
      Drop support for Python3.3
      website: link to PyPI project page w/o version; reflect current state of installation and development
      v0.24.0 release
      v0.24.1 release
      ssl: connect used non-monotonic time.time() for timeout (#520)
      New benchmarks runner
      greenthread: optimize _exit_funcs getattr/del dance; Thanks to Alex Kashirin
      IMPORTANT: late import in `use_hub()` + thread race caused using epolls even when it is unsupported on current platform
      maintainers list

Sourabh Deshmukh (1):
      typo

Stefan Nica (1):
      wsgi: handle remote connection resets

Tim Burke (10):
      wsgi: Don't strip all Unicode whitespace from headers on py3 (#504)
      wsgi: Use byte strings on py2 and unicode strings on py3
      wsgi: Catch and swallow IOErrors during discard() (#532)
      wsgi: Stop replacing invalid UTF-8 on py3
      wsgi: fix Input.readline on Python 3
      wsgi: fix Input.readlines when dealing with chunked input
      wsgi: Return 400 on negative Content-Length request headers (#537)
      wsgi: Only send 100 Continue response if no response has been sent yet (#557)
      wsgi: minimize API changes for 100-continue fix (#569)
      v0.25.0 release

Yuichi Bando (1):
      New feature: Add zipkin tracing to eventlet

costasgambit (1):
      websocket: support permessage-deflate extension; Thanks to Costas Christofi and Peter Kovary

jaimefrites (1):
      green.select: fix mark_as_closed() wrong number of args

nat-goodspeed (5):
      greendns: full comment lines were not skipped; Thanks to nat-goodspeed
      external dependencies for six, monotonic, dnspython
      Issue 535: use Python 2 compatible syntax for keyword-only args. (#536)
      Increase Travis slop factor for ZMQ CPU usage.  (#542)
      #53: Make a GreenPile with no spawn()s an empty sequence. (#555)

orishoshan (1):
      Issue #405: GreenSocket.accept does not notify_open (#406)

talwrii (1):
      green.zmq: support RCVTIMEO (receive timeout)

0.25.1
* wsgi (tests): Stop using deprecated cgi.parse_qs() to support Python 3.8; Thanks to Miro Hrončok
* os: Add workaround to `open` for pathlib on py 3.7; Thanks to David Szotten

0.25.0
======
* wsgi: Only send 100 Continue response if no response has been sent yet; Thanks to Tim Burke
* wsgi: Return 400 on negative Content-Length request headers; Thanks to Tim Burke
* Make a GreenPile with no spawn()s an empty sequence; Thanks to nat-goodspeed
* wsgi: fix Input.readlines when dealing with chunked input; Thanks to Tim Burke
* wsgi: fix Input.readline on Python 3; Thanks to Tim Burke
* wsgi: Stop replacing invalid UTF-8 on py3; Thanks to Tim Burke
* ssl: Fix compatibility with Python 3.7 ssl.SSLSocket; Thanks to Junyi
* reimport submodule as well in patcher.inject; Thanks to Junyi
* use Python 2 compatible syntax for keyword-only args; Thanks to nat-goodspeed

(NEWS truncated at 15 lines)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants