Skip to content

Commit

Permalink
Do not retry request on EADDRNOTAVAIL.
Browse files Browse the repository at this point in the history
This errno is caused by IPv6 configuration issues related to Docker.
Since skupperproject#401 fixes this, we can remove this workaround.
  • Loading branch information
jiridanek committed May 13, 2022
1 parent e43f50f commit 446c40e
Showing 1 changed file with 1 addition and 2 deletions.
3 changes: 1 addition & 2 deletions tests/system_tests_http.py
Original file line number Diff line number Diff line change
Expand Up @@ -97,8 +97,7 @@ def is_get_request_failing(self, url, use_ca=True, use_get_cert=False):
self.get(url, use_ca=use_ca)
return False
except OSError as e:
# EADDRNOTAVAIL happens in Docker when connecting to localhost:port where nobody listens
expected = (errno.ECONNREFUSED, errno.ECONNRESET, errno.EADDRNOTAVAIL)
expected = (errno.ECONNREFUSED, errno.ECONNRESET)
if e.errno in expected or e.reason.errno in expected:
return True
raise e
Expand Down

0 comments on commit 446c40e

Please sign in to comment.