Merge pull request #783 from DataDog/0.19-dev

0.19.0

Author: brettlangdon

.circleci/config.yml

@@ -51,6 +51,25 @@ jobs:
             - flake8.results
       - *save_cache_step
 
+  # Test that we can build the package properly and package long description will render
+  test_build:
+    docker:
+      - *test_runner
+    resource_class: *resource_class
+    steps:
+      - checkout
+      - *restore_cache_step
+      # Install required dependencies
+      - run: pip install twine readme_renderer[md]
+      # Ensure we didn't cache from previous runs
+      - run: rm -rf dist/
+      # Ensure package will build
+      - run: python setup.py sdist
+      # Ensure package long description is valid and will render
+      # https://github.com/pypa/twine/tree/6c4d5ecf2596c72b89b969ccc37b82c160645df8#twine-check
+      - run: twine check dist/*
+      - *save_cache_step
+
   tracer:
     docker:
       - *test_runner
@@ -931,11 +950,11 @@ workflows:
           <<: *deploy_docs_filters
           requires:
             - approve_docs_deployment
-
   test:
     jobs:
       - build_docs
       - flake8
+      - test_build
       - aiobotocore:
           requires:
             - flake8
@@ -1076,8 +1095,12 @@ workflows:
             - flake8
       - wait_all_tests:
           requires:
+            # Initial jobs
             - build_docs
             - flake8
+            - test_build
+
+            # flake8 dependent jobs
             - aiobotocore
             - aiohttp
             - aiopg

ddtrace/__init__.py

@@ -4,7 +4,7 @@
 from .tracer import Tracer
 from .settings import config
 
-__version__ = '0.18.0'
+__version__ = '0.19.0'
 
 # a global tracer instance with integration settings
 tracer = Tracer()

ddtrace/constants.py

@@ -1,3 +1,6 @@
 FILTERS_KEY = 'FILTERS'
-SAMPLE_RATE_METRIC_KEY = "_sample_rate"
+SAMPLE_RATE_METRIC_KEY = '_sample_rate'
 SAMPLING_PRIORITY_KEY = '_sampling_priority_v1'
+EVENT_SAMPLE_RATE_KEY = '_dd1.sr.eausr'
+
+NUMERIC_TAGS = (EVENT_SAMPLE_RATE_KEY, )

ddtrace/contrib/aiohttp/middlewares.py

@@ -1,10 +1,11 @@
 import asyncio
 
 from ..asyncio import context_provider
-from ...ext import AppTypes, http
 from ...compat import stringify
-from ...context import Context
+from ...constants import EVENT_SAMPLE_RATE_KEY
+from ...ext import AppTypes, http
 from ...propagation.http import HTTPPropagator
+from ...settings import config
 
 
 CONFIG_KEY = 'datadog_trace'
@@ -45,6 +46,10 @@ def attach_context(request):
             span_type=http.TYPE,
         )
 
+        # Configure trace search sample rate
+        if config.aiohttp.event_sample_rate is not None:
+            request_span.set_tag(EVENT_SAMPLE_RATE_KEY, config.aiohttp.event_sample_rate)
+
         # attach the context and the root span to the request; the Context
         # may be freely used by the application code
         request[REQUEST_CONTEXT_KEY] = request_span.context

ddtrace/contrib/bottle/trace.py

@@ -6,7 +6,9 @@
 from ddtrace.ext import http, AppTypes
 
 # project
+from ...constants import EVENT_SAMPLE_RATE_KEY
 from ...propagation.http import HTTPPropagator
+from ...settings import config
 
 SPAN_TYPE = 'web'
 
@@ -41,6 +43,10 @@ def wrapped(*args, **kwargs):
                     self.tracer.context_provider.activate(context)
 
             with self.tracer.trace('bottle.request', service=self.service, resource=resource, span_type=SPAN_TYPE) as s:
+                # Configure trace search sample rate
+                if config.bottle.event_sample_rate is not None:
+                    s.set_tag(EVENT_SAMPLE_RATE_KEY, config.bottle.event_sample_rate)
+
                 code = 0
                 try:
                     return callback(*args, **kwargs)

ddtrace/contrib/dbapi/__init__.py

@@ -8,9 +8,15 @@
 
 from ddtrace import Pin
 from ddtrace.ext import AppTypes, sql
+from ddtrace.settings import config
+from ddtrace.utils.formats import asbool, get_env
 
 log = logging.getLogger(__name__)
 
+config._add('dbapi2', dict(
+    trace_fetch_methods=asbool(get_env('dbapi2', 'trace_fetch_methods', 'false')),
+))
+
 
 class TracedCursor(wrapt.ObjectProxy):
     """ TracedCursor wraps a psql cursor and traces it's queries. """
@@ -72,6 +78,27 @@ def execute(self, query, *args, **kwargs):
         self._trace_method(self.__wrapped__.execute, self._self_datadog_name, query, {}, query, *args, **kwargs)
         return self
 
+    def callproc(self, proc, args):
+        """ Wraps the cursor.callproc method"""
+        self._self_last_execute_operation = proc
+        return self._trace_method(self.__wrapped__.callproc, self._self_datadog_name, proc, {}, proc, args)
+
+    def __enter__(self):
+        # previous versions of the dbapi didn't support context managers. let's
+        # reference the func that would be called to ensure that errors
+        # messages will be the same.
+        self.__wrapped__.__enter__
+
+        # and finally, yield the traced cursor.
+        return self
+
+
+class FetchTracedCursor(TracedCursor):
+    """
+    Sub-class of :class:`TracedCursor` that also instruments `fetchone`, `fetchall`, and `fetchmany` methods.
+
+    We do not trace these functions by default since they can get very noisy (e.g. `fetchone` with 100k rows).
+    """
     def fetchone(self, *args, **kwargs):
         """ Wraps the cursor.fetchone method"""
         span_name = '{}.{}'.format(self._self_datadog_name, 'fetchone')
@@ -101,25 +128,18 @@ def fetchmany(self, *args, **kwargs):
         return self._trace_method(self.__wrapped__.fetchmany, span_name, self._self_last_execute_operation, extra_tags,
                                   *args, **kwargs)
 
-    def callproc(self, proc, args):
-        """ Wraps the cursor.callproc method"""
-        self._self_last_execute_operation = proc
-        return self._trace_method(self.__wrapped__.callproc, self._self_datadog_name, proc, {}, proc, args)
-
-    def __enter__(self):
-        # previous versions of the dbapi didn't support context managers. let's
-        # reference the func that would be called to ensure that errors
-        # messages will be the same.
-        self.__wrapped__.__enter__
-
-        # and finally, yield the traced cursor.
-        return self
-
 
 class TracedConnection(wrapt.ObjectProxy):
     """ TracedConnection wraps a Connection with tracing code. """
 
-    def __init__(self, conn, pin=None, cursor_cls=TracedCursor):
+    def __init__(self, conn, pin=None, cursor_cls=None):
+        # Set default cursor class if one was not provided
+        if not cursor_cls:
+            # Do not trace `fetch*` methods by default
+            cursor_cls = TracedCursor
+            if config.dbapi2.trace_fetch_methods:
+                cursor_cls = FetchTracedCursor
+
         super(TracedConnection, self).__init__(conn)
         name = _get_vendor(conn)
         self._self_datadog_name = '{}.connection'.format(name)

ddtrace/contrib/django/middleware.py

@@ -4,9 +4,11 @@
 from .conf import settings
 from .compat import user_is_authenticated, get_resolver
 
-from ...ext import http
+from ...constants import EVENT_SAMPLE_RATE_KEY
 from ...contrib import func_name
+from ...ext import http
 from ...propagation.http import HTTPPropagator
+from ...settings import config
 
 # 3p
 from django.core.exceptions import MiddlewareNotUsed
@@ -118,6 +120,10 @@ def process_request(self, request):
                 span_type=http.TYPE,
             )
 
+            # Configure trace search sample rate
+            if config.django.event_sample_rate is not None:
+                span.set_tag(EVENT_SAMPLE_RATE_KEY, config.django.event_sample_rate)
+
             span.set_tag(http.METHOD, request.method)
             span.set_tag(http.URL, request.path)
             _set_req_span(request, span)

ddtrace/contrib/falcon/middleware.py

@@ -3,7 +3,9 @@
 from ddtrace.ext import http as httpx
 from ddtrace.http import store_request_headers, store_response_headers
 from ddtrace.propagation.http import HTTPPropagator
+
 from ...compat import iteritems
+from ...constants import EVENT_SAMPLE_RATE_KEY
 from ...ext import AppTypes
 from ...settings import config
 
@@ -37,6 +39,10 @@ def process_request(self, req, resp):
             span_type=httpx.TYPE,
         )
 
+        # Configure trace search sample rate
+        if config.falcon.event_sample_rate is not None:
+            span.set_tag(EVENT_SAMPLE_RATE_KEY, config.falcon.event_sample_rate)
+
         span.set_tag(httpx.METHOD, req.method)
         span.set_tag(httpx.URL, req.url)
 

ddtrace/contrib/flask/patch.py

@@ -7,6 +7,7 @@
 
 from ddtrace import config, Pin
 
+from ...constants import EVENT_SAMPLE_RATE_KEY
 from ...ext import AppTypes
 from ...ext import http
 from ...propagation.http import HTTPPropagator
@@ -284,6 +285,10 @@ def traced_wsgi_app(pin, wrapped, instance, args, kwargs):
     # We will override this below in `traced_dispatch_request` when we have a `RequestContext` and possibly a url rule
     resource = u'{} {}'.format(request.method, request.path)
     with pin.tracer.trace('flask.request', service=pin.service, resource=resource, span_type=http.TYPE) as s:
+        # Configure trace search sample rate
+        if config.flask.event_sample_rate is not None:
+            s.set_tag(EVENT_SAMPLE_RATE_KEY, config.flask.event_sample_rate)
+
         s.set_tag(FLASK_VERSION, flask_version_str)
 
         # Wrap the `start_response` handler to extract response code

ddtrace/contrib/molten/patch.py

@@ -4,6 +4,7 @@
 import molten
 
 from ... import Pin, config
+from ...constants import EVENT_SAMPLE_RATE_KEY
 from ...ext import AppTypes, http
 from ...propagation.http import HTTPPropagator
 from ...utils.formats import asbool, get_env
@@ -83,6 +84,10 @@ def patch_app_call(wrapped, instance, args, kwargs):
             pin.tracer.context_provider.activate(context)
 
     with pin.tracer.trace('molten.request', service=pin.service, resource=resource) as span:
+        # Configure trace search sample rate
+        if config.molten.event_sample_rate is not None:
+            span.set_tag(EVENT_SAMPLE_RATE_KEY, config.molten.event_sample_rate)
+
         @wrapt.function_wrapper
         def _w_start_response(wrapped, instance, args, kwargs):
             """ Patch respond handling to set metadata """