fix(tracer): wrap asyncio only when active trace context (#13326)

See linked issue #8907 and
release note for details.


## Checklist
- [x] PR author has checked that all the criteria below are met
- The PR description includes an overview of the change
- The PR description articulates the motivation for the change
- The change includes tests OR the PR description describes a testing
strategy
- The PR description notes risks associated with the change, if any
- Newly-added code is easy to change
- The change follows the [library release note
guidelines](https://ddtrace.readthedocs.io/en/stable/releasenotes.html)
- The change includes or references documentation updates if necessary
- Backport labels are set (if
[applicable](https://ddtrace.readthedocs.io/en/latest/contributing.html#backporting))

## Reviewer Checklist
- [x] Reviewer has checked that all the criteria below are met 
- Title is accurate
- All changes are related to the pull request's stated goal
- Avoids breaking
[API](https://ddtrace.readthedocs.io/en/stable/versioning.html#interfaces)
changes
- Testing strategy adequately addresses listed risks
- Newly-added code is easy to change
- Release note makes sense to a user of the library
- If necessary, author has acknowledged and discussed the performance
implications of this PR as reported in the benchmarks PR comment
- Backport labels are set in a manner that is consistent with the
[release branch maintenance
policy](https://ddtrace.readthedocs.io/en/latest/contributing.html#backporting)

Author: taegyunkim

ddtrace/contrib/internal/asyncio/patch.py

@@ -20,7 +20,7 @@ def patch():
         return
     asyncio._datadog_patch = True
     Pin().onto(asyncio)
-    wrap(asyncio.BaseEventLoop.create_task, _wrapped_create_task_py37)
+    wrap(asyncio.BaseEventLoop.create_task, _wrapped_create_task)
 
 
 def unpatch():
@@ -29,23 +29,29 @@ def unpatch():
     if getattr(asyncio, "_datadog_patch", False):
         return
     asyncio._datadog_patch = False
-    unwrap(asyncio.BaseEventLoop.create_task, _wrapped_create_task_py37)
+    unwrap(asyncio.BaseEventLoop.create_task, _wrapped_create_task)
 
 
-def _wrapped_create_task_py37(wrapped, args, kwargs):
+def _wrapped_create_task(wrapped, args, kwargs):
     """This function ensures the current active trace context is propagated to scheduled tasks.
     By default the trace context is propagated when a task is executed and NOT when it is created.
     """
     pin = Pin.get_from(asyncio)
     if not pin or not pin.enabled():
         return wrapped(*args, **kwargs)
 
-    # override existing co-rountine to ensure the current trace context is propagated
-    coro = get_argument_value(args, kwargs, 1, "coro")
+    # Get current trace context
     dd_active = pin.tracer.current_trace_context()
+    # Only wrap the coroutine if we have an active trace context
+    if not dd_active:
+        return wrapped(*args, **kwargs)
+
+    # Get the coroutine
+    coro = get_argument_value(args, kwargs, 1, "coro")
 
+    # Wrap the coroutine and ensure the current trace context is propagated
     async def traced_coro(*args_c, **kwargs_c):
-        if dd_active and dd_active != pin.tracer.current_trace_context():
+        if dd_active != pin.tracer.current_trace_context():
             pin.tracer.context_provider.activate(dd_active)
         return await coro
 

releasenotes/notes/tracer-asyncio-create_task-wrap-3d171cb8429be8a7.yaml

@@ -0,0 +1,7 @@
+---
+fixes:
+  - |
+    tracer: fixes ``RuntimeWarning`` from an unwaited coroutine during tab
+    completion in IPython REPL when ``asyncio`` integration is active. Tracer
+    now wraps an ``asyncio`` coroutine only when  there is an active trace
+    context.

tests/contrib/suitespec.yml

@@ -16,6 +16,7 @@ components:
     - ddtrace/contrib/internal/asgi/*
   asyncio:
     - ddtrace/contrib/_asyncio.py
+    - ddtrace/contrib/internal/asyncio/*
   avro:
     - ddtrace/contrib/_avro.py
     - ddtrace/contrib/internal/avro/*