]> review.fuel-infra Code Review - openstack-build/heat-build.git/commitdiff
Remove support for eventlet 0.9.16
authorDirk Mueller <dirk@dmllr.de>
Sun, 4 Aug 2013 15:49:40 +0000 (17:49 +0200)
committerDirk Mueller <dirk@dmllr.de>
Sun, 4 Aug 2013 15:49:40 +0000 (17:49 +0200)
We require 0.12.0 or newer, so 0.9.16 compatibility code
is no longer needed.

Change-Id: Idda955b64ce625ea30cc18366df01cc024cf26d1

heat/common/wsgi.py

index f108fe68e5b8db6b7e4fa67bf292ecf8975c0cb8..92d7a1958c39f94139945d1a534ce03c27b29a34 100644 (file)
@@ -49,10 +49,6 @@ from heat.openstack.common.gettextutils import _
 
 URL_LENGTH_LIMIT = 50000
 
-# TODO(shadower) remove this once eventlet 0.9.17 is in distros Heat
-# supports (notably Fedora 17 and Ubuntu 12.04 and newer)
-eventlet.wsgi.MAX_REQUEST_LINE = URL_LENGTH_LIMIT
-
 bind_opts = [
     cfg.StrOpt('bind_host', default='0.0.0.0'),
     cfg.IntOpt('bind_port'),
@@ -246,7 +242,7 @@ class Server(object):
         eventlet.patcher.monkey_patch(all=False, socket=True)
         self.pool = eventlet.GreenPool(size=self.threads)
         try:
-            eventlet_wsgi_server(self.sock,
+            eventlet.wsgi.server(self.sock,
                                  self.application,
                                  custom_pool=self.pool,
                                  url_length_limit=URL_LENGTH_LIMIT,
@@ -259,25 +255,12 @@ class Server(object):
     def _single_run(self, application, sock):
         """Start a WSGI server in a new green thread."""
         self.logger.info(_("Starting single process server"))
-        eventlet_wsgi_server(sock, application,
+        eventlet.wsgi.server(sock, application,
                              custom_pool=self.pool,
                              url_length_limit=URL_LENGTH_LIMIT,
                              log=WritableLogger(self.logger))
 
 
-def eventlet_wsgi_server(sock, application, **kwargs):
-    '''
-    Return a new instance of the eventlet wsgi server with the proper url limit
-    in a way that's compatible with eventlet 0.9.16 and 0.9.17.
-    '''
-    try:
-        return eventlet.wsgi.server(sock, application, **kwargs)
-    # TODO(shadower) remove this when we don't support eventlet 0.9.16 anymore
-    except TypeError:
-        kwargs.pop('url_length_limit', None)
-        return eventlet.wsgi.server(sock, application, **kwargs)
-
-
 class Middleware(object):
     """
     Base WSGI middleware wrapper. These classes require an application to be