]> review.fuel-infra Code Review - openstack-build/neutron-build.git/commitdiff
Revert "monkey patch stdlib before importing other modules"
authorarmando-migliaccio <armamig@gmail.com>
Thu, 12 Feb 2015 01:25:56 +0000 (17:25 -0800)
committerarmando-migliaccio <armamig@gmail.com>
Thu, 12 Feb 2015 01:26:33 +0000 (17:26 -0800)
The projects that depend on Neutron do not play nicely with this.

This reverts commit 24b11ded7f6ff12f9484aecdb8d30498bf1b9025.

Change-Id: Ic7afd7de9db97cc3a085bc842a2d990519cf42f6

neutron/__init__.py
run_tests.sh
tox.ini

index 20549b760a8eb3df5a90f139aab2dea98cd61c50..710b18c464114e5a14425819bed11b26cce6962a 100644 (file)
 #    under the License.
 
 import gettext
-import os
-
-if not os.environ.get('NO_EVENTLET'):
-    import eventlet
-    eventlet.monkey_patch()
 
 
 gettext.install('neutron', unicode=1)
index b6224f07ce7aeba93f2d7489ef7685fdd19fc4d6..6fe001f5381d5652c64ff20cf292d4fcfab667d3 100755 (executable)
@@ -179,7 +179,7 @@ function warn_on_flake8_without_venv {
 function run_pep8 {
   echo "Running flake8 ..."
   warn_on_flake8_without_venv
-  NO_EVENTLET=1 ${wrapper} flake8
+  ${wrapper} flake8
 }
 
 function run_pep8_changed {
@@ -192,7 +192,7 @@ function run_pep8_changed {
     local files=$(git diff --name-only $target | tr '\n' ' ')
     echo "Running flake8 on ${files}"
     warn_on_flake8_without_venv
-    diff -u --from-file /dev/null ${files} | NO_EVENTLET=1 ${wrapper} flake8 --diff
+    diff -u --from-file /dev/null ${files} | ${wrapper} flake8 --diff
 }
 
 
diff --git a/tox.ini b/tox.ini
index 9d88fd4942a46fd096970580d6a8ecaac38a3fd4..3d1f62f44fd0618b7eb588d8d9f33f976b245745 100644 (file)
--- a/tox.ini
+++ b/tox.ini
@@ -49,7 +49,6 @@ sitepackages = True
 downloadcache = ~/cache/pip
 
 [testenv:pep8]
-setenv = NO_EVENTLET=1
 commands =
   # If it is easier to add a check via a shell script, consider adding it in this file
   sh ./tools/misc-sanity-checks.sh