Skip to content

Conversation

@dubloom
Copy link
Contributor

@dubloom dubloom commented Nov 12, 2025

This PR implements this RFC for telemetry.

Add process_tags to telemetry configuration

Testing

  • There is already a test testing the exact config so we know that if we are not enabling the process tags, it is not in the config
  • Check process tags are in the telemetry configuration when enabled.

@github-actions
Copy link
Contributor

CODEOWNERS have been resolved as:

ddtrace/internal/telemetry/data.py                                      @DataDog/apm-python
tests/telemetry/test_data.py                                            @DataDog/apm-python

@dubloom dubloom added the changelog/no-changelog A changelog entry is not required for this PR. label Nov 12, 2025
@github-actions
Copy link
Contributor

github-actions bot commented Nov 12, 2025

Bootstrap import analysis

Comparison of import times between this PR and base.

Summary

The average import time from this PR is: 238 ± 2 ms.

The average import time from base is: 241 ± 3 ms.

The import time difference between this PR and base is: -3.2 ± 0.1 ms.

Import time breakdown

The following import paths have shrunk:

ddtrace.auto 2.091 ms (0.88%)
ddtrace.bootstrap.sitecustomize 1.408 ms (0.59%)
ddtrace.bootstrap.preload 1.408 ms (0.59%)
ddtrace.internal.remoteconfig.client 0.653 ms (0.28%)
ddtrace 0.683 ms (0.29%)
ddtrace.internal._unpatched 0.029 ms (0.01%)
json 0.029 ms (0.01%)
json.decoder 0.029 ms (0.01%)
re 0.029 ms (0.01%)
enum 0.029 ms (0.01%)
types 0.029 ms (0.01%)

@dubloom dubloom force-pushed the dubloom/process-tags-telemetry branch from 7e097ee to 256f2de Compare November 12, 2025 15:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changelog/no-changelog A changelog entry is not required for this PR.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants