From c34e11a2f85293855c8645a786794ac9eea6f4bf Mon Sep 17 00:00:00 2001 From: Jordan Pittier Date: Fri, 10 Feb 2017 17:03:19 +0100 Subject: [PATCH] Remove dependency on memory heavy python-requests. It turns out that Neutron uses python-requests on just one place and for no good reason. Given that loading Requests amount for 80MB of RAM in my system, maybe it's worth removing our dependency on requests. Change-Id: I905252a55bd174ff1aa8bdec42fe32432746846f --- neutron/agent/l3/keepalived_state_change.py | 3 +-- requirements.txt | 1 - 2 files changed, 1 insertion(+), 3 deletions(-) diff --git a/neutron/agent/l3/keepalived_state_change.py b/neutron/agent/l3/keepalived_state_change.py index 040a391b4fe..e63b707f9a3 100644 --- a/neutron/agent/l3/keepalived_state_change.py +++ b/neutron/agent/l3/keepalived_state_change.py @@ -20,7 +20,6 @@ import httplib2 import netaddr from oslo_config import cfg from oslo_log import log as logging -import requests from neutron._i18n import _, _LE from neutron.agent.l3 import ha @@ -105,7 +104,7 @@ class MonitorDaemon(daemon.Daemon): 'X-Neutron-State': state}, connection_type=KeepalivedUnixDomainConnection) - if resp.status != requests.codes.ok: + if resp.status != 200: raise Exception(_('Unexpected response: %s') % resp) LOG.debug('Notified agent router %s, state %s', self.router_id, state) diff --git a/requirements.txt b/requirements.txt index 2e9ac43d12b..db56d43befa 100644 --- a/requirements.txt +++ b/requirements.txt @@ -11,7 +11,6 @@ debtcollector>=1.2.0 # Apache-2.0 eventlet!=0.18.3,>=0.18.2 # MIT pecan!=1.0.2,!=1.0.3,!=1.0.4,!=1.2,>=1.0.0 # BSD httplib2>=0.7.5 # MIT -requests!=2.12.2,>=2.10.0 # Apache-2.0 Jinja2!=2.9.0,!=2.9.1,!=2.9.2,!=2.9.3,!=2.9.4,>=2.8 # BSD License (3 clause) keystonemiddleware>=4.12.0 # Apache-2.0 netaddr!=0.7.16,>=0.7.13 # BSD