openstack/neutron
Code Issues Proposed changes

Merge "Fix pylint "W" missing-timeout warnings"

Browse Source
This commit is contained in:
Zuul 2023-12-02 00:55:55 +00:00 committed by Gerrit Code Review
commit 0fa45c8684
5 changed files with 8 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
.pylintrc
View File

@ -57,7 +57,6 @@ disable=
arguments-renamed,
broad-exception-raised,
unspecified-encoding,
missing-timeout,
redundant-u-string-prefix,
unused-private-member,
# "C" Coding convention violations

3
neutron/agent/metadata/agent.py
View File

@ -256,7 +256,8 @@ class MetadataProxyHandler(object):
headers=headers,
data=req.body,
cert=client_cert,
verify=verify_cert)
verify=verify_cert,
timeout=60)
if resp.status_code == 200:
req.response.content_type = resp.headers['content-type']

3
neutron/agent/ovn/metadata/server.py
View File

@ -153,7 +153,8 @@ class MetadataProxyHandler(object):
headers=headers,
data=req.body,
cert=client_cert,
verify=verify_cert)
verify=verify_cert,
timeout=60)
if resp.status_code == 200:
req.response.content_type = resp.headers['content-type']

3
neutron/tests/unit/agent/metadata/test_agent.py
View File

@ -434,7 +434,8 @@ class _TestMetadataProxyHandlerCacheMixin(object):
data=body,
cert=(self.fake_conf.nova_client_cert,
self.fake_conf.nova_client_priv_key),
verify=False)
verify=False,
timeout=60)
return retval

3
neutron/tests/unit/agent/ovn/metadata/test_server.py
View File

@ -167,7 +167,8 @@ class TestMetadataProxyHandler(base.BaseTestCase):
data=body,
cert=(self.fake_conf.nova_client_cert,
self.fake_conf.nova_client_priv_key),
verify=False)
verify=False,
timeout=60)
return retval