openstack/heat
Code Issues Proposed changes

Merge "Use err.errno to access errno for socket errors"

Browse Source
This commit is contained in:
Zuul
2020-10-01 20:26:41 +00:00
committed by Gerrit Code Review
parent 8a0736e0ea a78ddafdec
commit 33dc1129f1
2 changed files with 5 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
heat/common/wsgi.py
View File

@@ -263,7 +263,7 @@ def get_socket(conf, default_port):
backlog=conf.backlog,
family=address_family)
except socket.error as err:
if err.args[0] != errno.EADDRINUSE:
if err.errno != errno.EADDRINUSE:
raise
eventlet.sleep(0.1)
if not sock:
@@ -584,7 +584,7 @@ class Server(object):
keepalive=cfg.CONF.eventlet_opts.wsgi_keep_alive,
socket_timeout=socket_timeout)
except socket.error as err:
if err[0] != errno.EINVAL:
if err.errno != errno.EINVAL:
raise
self.pool.waitall()

4
heat/tests/api/test_wsgi.py
View File

@@ -441,10 +441,12 @@ class GetSocketTestCase(common.HeatTestCase):
wsgi.cfg.CONF.heat_api, 1234)
def test_get_socket_with_bind_problems(self):
err = wsgi.socket.error(
socket.errno.EADDRINUSE, 'Address already in use')
self.useFixture(fixtures.MonkeyPatch(
'heat.common.wsgi.eventlet.listen',
mock.Mock(side_effect=(
[wsgi.socket.error(socket.errno.EADDRINUSE)] * 3 + [None]))))
[err] * 3 + [None]))))
self.useFixture(fixtures.MonkeyPatch(
'heat.common.wsgi.ssl.wrap_socket',
lambda *x, **y: None))