From 1c1bb732021a318143a4a5eef461fdbe2ccac42e Mon Sep 17 00:00:00 2001 From: Brian Haley Date: Wed, 12 Sep 2018 10:54:47 -0600 Subject: [PATCH] Fix pylint warning with eventlet > 0.22.0 When eventlet was bumped up to 0.24.1 in upper-constraints [1] it started generating a pylint warning, since neutron has support for older versions such as 0.22.0 that take a different number of arguments. Just silence the warning. [1] https://review.openstack.org/#/c/589382/ This warning can also happen on 0.20 packages with backport fixes Change-Id: I169ecb6f33b670c5469919214a32645cd9eda206 Closes-Bug: #1791178 (cherry picked from commit db7e19d2ba3fcde0510be0094a4851178fae578b) --- neutron/agent/linux/utils.py | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/neutron/agent/linux/utils.py b/neutron/agent/linux/utils.py index 79fbf53fa12..b3fae1e42d5 100644 --- a/neutron/agent/linux/utils.py +++ b/neutron/agent/linux/utils.py @@ -405,6 +405,10 @@ class UnixDomainHttpProtocol(eventlet.wsgi.HttpProtocol): if not client_address: client_address = ('', 0) # base class is old-style, so super does not work properly + # NOTE: eventlet 0.22 or later changes the number of args to 2. + # If we install eventlet 0.22 or later into a venv for pylint, + # pylint complains this. Let's skip it. (bug 1791178) + # pylint: disable=too-many-function-args eventlet.wsgi.HttpProtocol.__init__( self, request, client_address, server) else: