When we updated the ThreadPoolExecutor implementation to pass Context instead of Span one piece of information we are not passing along is the current Span's "service".
We do not need this since spans created within the ThreadPoolExecutor will use the DD_SERVICE that is set.
These tests do not set a DD_SERVICE so it is expected that their service names are None (will inherit from their parent).
Checklist
[x] Change(s) are motivated and described in the PR description
[x] Testing strategy is described if automated tests are not included in the PR
[x] Risks are described (performance impact, potential for breakage, maintainability)
[x] Change is maintainable (easy to change, telemetry, documentation)
Fix failing tornado jobs.
When we updated the
ThreadPoolExecutor
implementation to passContext
instead ofSpan
one piece of information we are not passing along is the current Span's "service".We do not need this since spans created within the
ThreadPoolExecutor
will use theDD_SERVICE
that is set.These tests do not set a
DD_SERVICE
so it is expected that their service names areNone
(will inherit from their parent).Checklist
changelog/no-changelog
is set@DataDog/apm-tees
.Reviewer Checklist