fix(asyncio): avoid overwriting coroutine names

[mabdinur]

Resolves: #10975

Checklist

• 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
• The change includes or references documentation updates if necessary
• Backport labels are set (if applicable)

Reviewer Checklist

• 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 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

[github-actions]

CODEOWNERS have been resolved as:

releasenotes/notes/avoid-overriding-couroutine-names-0852ea19200feb74.yaml  @DataDog/apm-python
ddtrace/contrib/internal/asyncio/patch.py                               @DataDog/apm-core-python @DataDog/apm-idm-python
tests/contrib/asyncio/test_tracer.py                                    @DataDog/apm-core-python @DataDog/apm-idm-python

[datadog-dd-trace-py-rkomorn]

Datadog Report

Branch report: munir/avoid-overriding-couroutine-names
Commit report: 5358ffc
Test service: dd-trace-py

✅ 0 Failed, 130 Passed, 1378 Skipped, 4m 26.95s Total duration (35m 2.26s time saved)

[pr-commenter]

Benchmarks

Benchmark execution time: 2025-01-24 01:39:47

Comparing candidate commit fa3e04f in PR branch munir/avoid-overriding-couroutine-names with baseline commit 731be65 in branch main.

Found 1 performance improvements and 1 performance regressions! Performance is the same for 392 metrics, 2 unstable metrics.

scenario:iast_aspects-ljust_aspect

• 🟥 execution_time [+1.069µs; +1.170µs] or [+7.651%; +8.376%]

scenario:iast_aspects-ospathjoin_aspect

• 🟩 execution_time [-507.294ns; -444.886ns] or [-9.042%; -7.930%]

[P403n1x87]

Would wrapping with the wrapping context make for a simpler solution?

[mabdinur]

Would wrapping with the wrapping context make for a simpler solution?

Yup, it would. I avoided this because ddtrace.internal.wrapping.wrap(...) does not support wrapping coroutines. Coroutines do not have a __code__ attribute (it is likely implemented in C) and the analogous cr_code property is immutable (so we can't modify it). I am not sure how to work around this limitation 😞.

[datadog-dd-trace-py-rkomorn]

Datadog Report

Branch report: munir/avoid-overriding-couroutine-names
Commit report: fa3e04f
Test service: dd-trace-py

✅ 0 Failed, 130 Passed, 1378 Skipped, 5m 10.37s Total duration (35m 3.79s time saved)