feat(tracing): support listing headers in DD_TRACE_HEADER_TAGS without tag names

[mabdinur]

This will align the implementation of DD_TRACE_HEADER_TAGS with the golang and java tracers.

Checklist

• Change(s) are motivated and described in the PR description
• Testing strategy is described if automated tests are not included in the PR
• Risks are described (performance impact, potential for breakage, maintainability)
• Change is maintainable (easy to change, telemetry, documentation)
• Library release note guidelines are followed or label changelog/no-changelog is set
• Documentation is included (in-code, generated user docs, public corp docs)
• Backport labels are set (if applicable)
• If this PR changes the public interface, I've notified @DataDog/apm-tees.

Reviewer Checklist

• Title is accurate
• All changes are related to the pull request's stated goal
• Description motivates each change
• Avoids breaking API changes
• Testing strategy adequately addresses listed risks
• Change is maintainable (easy to change, telemetry, documentation)
• Release note makes sense to a user of the library
• 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

[datadog-dd-trace-py-rkomorn]

Datadog Report

Branch report: munir/update-header-tags
Commit report: da02228
Test service: dd-trace-py

❌ 62 Failed (0 Known Flaky), 144622 Passed, 31498 Skipped, 9h 40m 40.28s Total duration (1h 9m 45.04s time saved)

❌ Failed Tests (62)

This report shows up to 5 failed tests.

• test_get_malformed - test_http.py - Details

  Expand for error

   assert 8 == 7
    +  where 8 = len({'host': 'fc7337c02ef1', 'language': 'python', 'mytagfoobar': 'mytagfoobar', 'profiler_version': '2.11.0.dev12+gda0222868', ...})
  

• test_get_malformed - test_http.py - Details

  Expand for error

   assert 8 == 7
    +  where 8 = len({'host': '301b32856fee', 'language': 'python', 'mytagfoobar': 'mytagfoobar', 'profiler_version': '2.11.0.dev12+gda0222868', ...})
  

• test_get_malformed - test_http.py - Details

  Expand for error

   assert 8 == 7
    +  where 8 = len({'host': '653e1f94fb3b', 'language': 'python', 'mytagfoobar': 'mytagfoobar', 'profiler_version': '2.11.0.dev12+gda0222868', ...})
  

• test_get_malformed - test_http.py - Details

  Expand for error

   assert 8 == 7
    +  where 8 = len({'host': '7957ab02e76f', 'language': 'python', 'mytagfoobar': 'mytagfoobar', 'profiler_version': '2.11.0.dev12+gda0222868', ...})
  

• test_get_malformed - test_http.py

[mtoffl01]

Can you be sure to update this in the system tests as well? - https://github.com/DataDog/system-tests/blob/main/manifests/python.yml#L682

@mabdinur my link is outdated b/c the file changed, but it's the Test_HeaderTags_Short: missing_feature field

[datadog-dd-trace-py-rkomorn]

Datadog Report

Branch report: munir/update-header-tags
Commit report: 5668fce
Test service: dd-trace-py

❌ 2 Failed (0 Known Flaky), 144910 Passed, 31463 Skipped, 8h 11m 1.3s Total duration (1h 52m 52.11s time saved)

❌ Failed Tests (2)

• test_dd_tags_invalid - test_tracer.py - Details

  Expand for error

   Subprocess Test "python -m unittest tests.tracer.test_tracer.EnvTracerTestCase.test_dd_tags_invalid" Failed (exit code 1):
   F
   ======================================================================
   FAIL: test_dd_tags_invalid (tests.tracer.test_tracer.EnvTracerTestCase)
   ----------------------------------------------------------------------
   Traceback (most recent call last):
     File "/root/project/tests/tracer/test_tracer.py", line 987, in test_dd_tags_invalid
       assert self.tracer._tags.get("key3")
   AssertionError
   
   ...
  

• test_hourglass_turn - test_utils.py - Details

  Expand for error

   assert 0.36190004800005227 < 0.3
    +  where 0.36190004800005227 = <bound method StopWatch.elapsed of <ddtrace.internal.utils.time.StopWatch object at 0x7efd682e1c70>>()
    +    where <bound method StopWatch.elapsed of <ddtrace.internal.utils.time.StopWatch object at 0x7efd682e1c70>> = <ddtrace.internal.utils.time.StopWatch object at 0x7efd682e1c70>.elapsed
  

[pr-commenter]

Benchmarks

Benchmark execution time: 2024-07-24 18:43:26

Comparing candidate commit 165c988 in PR branch munir/update-header-tags with baseline commit 2de06b2 in branch main.

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

[codecov-commenter]

Codecov Report

Attention: Patch coverage is 0% with 23 lines in your changes missing coverage. Please review.

Project coverage is 10.54%. Comparing base (9cce458) to head (80aff99).

Files	Patch %	Lines
tests/tracer/test_trace_utils.py	0.00%	18 Missing ⚠️
ddtrace/internal/utils/formats.py	0.00%	3 Missing ⚠️
tests/tracer/test_tracer.py	0.00%	2 Missing ⚠️

Additional details and impacted files

@@             Coverage Diff             @@
##             main    #9589       +/-   ##
===========================================
- Coverage   73.93%   10.54%   -63.39%     
===========================================
  Files        1402     1368       -34     
  Lines      130420   127993     -2427     
===========================================
- Hits        96426    13502    -82924     
- Misses      33994   114491    +80497     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[mabdinur]

Done.

[github-actions]

CODEOWNERS have been resolved as:

releasenotes/notes/list_dd_header_tags-f8a9ce1daa2a1cf6.yaml            @DataDog/apm-python
ddtrace/contrib/trace_utils.py                                          @DataDog/apm-core-python @DataDog/apm-idm-python
ddtrace/internal/utils/formats.py                                       @DataDog/apm-core-python
docs/configuration.rst                                                  @DataDog/apm-core-python
tests/internal/test_settings.py                                         @DataDog/apm-core-python
tests/tracer/test_trace_utils.py                                        @DataDog/apm-sdk-api-python
tests/tracer/test_tracer.py                                             @DataDog/apm-sdk-api-python
tests/tracer/test_utils.py                                              @DataDog/apm-sdk-api-python

[datadog-dd-trace-py-rkomorn]

Datadog Report

Branch report: munir/update-header-tags
Commit report: 80aff99
Test service: dd-trace-py

✅ 0 Failed, 133542 Passed, 44232 Skipped, 6h 53m 9.08s Total duration (12m 3.16s time saved)

[datadog-datadog-prod-us1]

Library Vulnerabilities