Removing support for python 2.7 (#2)

OpenTelemetry doesn't support Python 2.7. Removing it from the tests and removing contrib packages that don't support Python 3. In this change:

- removing pylons contrib
- removing mysqldb contrib
- updating minimum versions of flask (>=1.0), gevent (>=1.1)

Signed-off-by: Alex Boten <aboten@lightstep.com>

.circleci/config.yml

@@ -202,17 +202,6 @@ jobs:
       - *persist_to_workspace_step
       - *save_cache_step
 
-  pylons:
-    docker:
-      - *test_runner
-    resource_class: *resource_class
-    steps:
-      - checkout
-      - *restore_cache_step
-      - run: scripts/run-tox-scenario '^pylons_contrib-'
-      - *persist_to_workspace_step
-      - *save_cache_step
-
   aiohttp:
     docker:
       - *test_runner
@@ -318,7 +307,7 @@ jobs:
     steps:
       - checkout
       - *restore_cache_step
-      - run: tox -e 'falcon_contrib{,_autopatch}-{py27,py34,py35,py36}-falcon{10,11,12,13,14}' --result-json /tmp/falcon.results
+      - run: tox -e 'falcon_contrib{,_autopatch}-{py34,py35,py36}-falcon{10,11,12,13,14}' --result-json /tmp/falcon.results
       - *persist_to_workspace_step
       - *save_cache_step
 
@@ -720,7 +709,7 @@ jobs:
       - run:
           command: |
             mkdir -p /tmp/test-reports
-            tox -e 'benchmarks-{py27,py34,py35,py36,py37}' --result-json /tmp/benchmarks.results -- --benchmark-storage=file:///tmp/test-reports/ --benchmark-autosave
+            tox -e 'benchmarks-{py34,py35,py36,py37}' --result-json /tmp/benchmarks.results -- --benchmark-storage=file:///tmp/test-reports/ --benchmark-autosave
       - store_test_results:
           path: /tmp/test-reports
       - store_artifacts:
@@ -985,10 +974,6 @@ workflows:
           requires:
             - flake8
             - black
-      - pylons:
-          requires:
-            - flake8
-            - black
       - pymemcache:
           requires:
             - flake8
@@ -1097,7 +1082,6 @@ workflows:
             - opentracer
             - psycopg
             - pylibmc
-            - pylons
             - pymemcache
             - pymongo
             - pymysql

conftest.py

@@ -11,7 +11,7 @@
 
 import pytest
 
-PY_DIR_PATTERN = re.compile(r"^py[23][0-9]$")
+PY_DIR_PATTERN = re.compile(r"^py3[0-9]$")
 
 
 # Determine if the folder should be ignored
@@ -27,7 +27,6 @@ def pytest_ignore_collect(path, config):
     Example::
 
         File: tests/contrib/vertica/py35/test.py
-        Python 2.7: Skip
         Python 3.4: Skip
         Python 3.5: Collect
         Python 3.6: Collect

ddtrace/bootstrap/sitecustomize.py

@@ -43,7 +43,6 @@
     "django": True,
     "falcon": True,
     "flask": True,
-    "pylons": True,
     "pyramid": True,
 }
 

ddtrace/commands/ddtrace_run.py

@@ -31,7 +31,7 @@
     DATADOG_TRACE_AGENT_PORT=8126: override the port that the default tracer will submit to (default: 8126)
     DATADOG_SERVICE_NAME : override the service name to be used for this program (no default)
                            This value is passed through when setting up middleware for web framework integrations.
-                           (e.g. pylons, flask, django)
+                           (e.g. flask, django)
                            For tracing without a web integration, prefer setting the service name in code.
     DATADOG_PRIORITY_SAMPLING=true|false : (default: false): enables Priority Sampling.
 """  # noqa: E501

ddtrace/contrib/flask/patch.py

@@ -156,13 +156,6 @@ def patch():
         for signal in signals:
             module = 'flask'
 
-            # v0.9 missed importing `appcontext_tearing_down` in `flask/__init__.py`
-            #  https://github.com/pallets/flask/blob/0.9/flask/__init__.py#L35-L37
-            #  https://github.com/pallets/flask/blob/0.9/flask/signals.py#L52
-            # DEV: Version 0.9 doesn't have a patch version
-            if flask_version <= (0, 9) and signal == 'appcontext_tearing_down':
-                module = 'flask.signals'
-
             # DEV: Patch `receivers_for` instead of `connect` to ensure we don't mess with `disconnect`
             _w(module, '{}.receivers_for'.format(signal), traced_signal_receivers_for(signal))
 
@@ -241,13 +234,6 @@ def unpatch():
         # Handle 'flask.request_started.receivers_for'
         obj = flask
 
-        # v0.9.0 missed importing `appcontext_tearing_down` in `flask/__init__.py`
-        #  https://github.com/pallets/flask/blob/0.9/flask/__init__.py#L35-L37
-        #  https://github.com/pallets/flask/blob/0.9/flask/signals.py#L52
-        # DEV: Version 0.9 doesn't have a patch version
-        if flask_version <= (0, 9) and prop == 'appcontext_tearing_down.receivers_for':
-            obj = flask.signals
-
         if '.' in prop:
             attr, _, prop = prop.partition('.')
             obj = getattr(obj, attr, object())

ddtrace/monkey.py

@@ -57,7 +57,6 @@
     # Ignore some web framework integrations that might be configured explicitly in code
     'django': False,
     'falcon': False,
-    'pylons': False,
     'pyramid': False,
 
     # Standard library modules off by default

docs/advanced_usage.rst

@@ -52,8 +52,6 @@ Supported web frameworks:
 +-------------------+---------+
 | :ref:`flask`      | True    |
 +-------------------+---------+
-| :ref:`pylons`     | True    |
-+-------------------+---------+
 | :ref:`pyramid`    | True    |
 +-------------------+---------+
 | :ref:`requests`   | True    |
@@ -197,7 +195,6 @@ Enabling APM events for all web frameworks can be accomplished by setting the en
 * :ref:`falcon`
 * :ref:`flask`
 * :ref:`molten`
-* :ref:`pylons`
 * :ref:`pyramid`
 * :ref:`requests`
 * :ref:`tornado`
@@ -238,8 +235,6 @@ For most libraries, APM events can be enabled with the environment variable ``DD
 +----------------------+----------------------------------------+
 | :ref:`pylibmc`       | ``DD_PYLIBMC_ANALYTICS_ENABLED``       |
 +----------------------+----------------------------------------+
-| :ref:`pylons`        | ``DD_PYLONS_ANALYTICS_ENABLED``        |
-+----------------------+----------------------------------------+
 | :ref:`pymemcache`    | ``DD_PYMEMCACHE_ANALYTICS_ENABLED``    |
 +----------------------+----------------------------------------+
 | :ref:`pymongo`       | ``DD_PYMONGO_ANALYTICS_ENABLED``       |
@@ -564,7 +559,7 @@ The available environment variables for ``ddtrace-run`` are:
   the tracer
 * ``DATADOG_SERVICE_NAME`` (no default): override the service name to be used
   for this program. This value is passed through when setting up middleware for
-  web framework integrations (e.g. pylons, flask, django). For tracing without a
+  web framework integrations (e.g. flask, django). For tracing without a
   web integration, prefer setting the service name in code.
 * ``DATADOG_PATCH_MODULES=module:patch,module:patch...`` e.g.
   ``boto:true,redis:false``: override the modules patched for this execution of