Skip to content

Conversation

@dubloom
Copy link
Contributor

@dubloom dubloom commented Nov 12, 2025

This PR implements this RFC for profiling.

Add process_tags to profiling tags

Testing

  • Check that process tags are not included in profiling tags if deactivated
  • Check the right process tags are added in profiling tags when activated

@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

CODEOWNERS have been resolved as:

ddtrace/profiling/profiler.py                                           @DataDog/profiling-python
tests/profiling/test_profiler.py                                        @DataDog/profiling-python
tests/profiling_v2/exporter/test_ddup.py                                @DataDog/profiling-python

@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: 205 ± 3 ms.

The average import time from base is: 210 ± 4 ms.

The import time difference between this PR and base is: -5.9 ± 0.2 ms.

Import time breakdown

The following import paths have shrunk:

ddtrace.auto 2.062 ms (1.01%)
ddtrace.bootstrap.sitecustomize 1.350 ms (0.66%)
ddtrace.bootstrap.preload 1.350 ms (0.66%)
ddtrace.internal.remoteconfig.client 0.603 ms (0.29%)
ddtrace 0.713 ms (0.35%)
ddtrace.internal._unpatched 0.148 ms (0.07%)
subprocess 0.107 ms (0.05%)
contextlib 0.107 ms (0.05%)
json 0.042 ms (0.02%)
json.decoder 0.042 ms (0.02%)
re 0.042 ms (0.02%)
enum 0.042 ms (0.02%)
types 0.042 ms (0.02%)
ddtrace._logger 0.098 ms (0.05%)
ddtrace.internal.telemetry 0.098 ms (0.05%)
ddtrace.internal.utils.formats 0.098 ms (0.05%)
ddtrace.internal.compat 0.098 ms (0.05%)
wrapt 0.098 ms (0.05%)
wrapt.importer 0.098 ms (0.05%)
importlib.metadata 0.098 ms (0.05%)

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