Skip to content

Conversation

@dubloom
Copy link
Contributor

@dubloom dubloom commented Nov 12, 2025

This PR implements this RFC for crashtracking.

Add process_tags to crashtracking payload

Testing

  • Check that process tags are not included if deactivated
  • Check the right process tags are set in the payload 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/internal/core/crashtracking.py                                  @DataDog/apm-core-python
tests/internal/crashtracker/test_crashtracker.py                        @DataDog/apm-core-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: 243 ± 5 ms.

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

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

Import time breakdown

The following import paths have shrunk:

ddtrace.auto 1.716 ms (0.71%)
ddtrace.bootstrap.sitecustomize 1.044 ms (0.43%)
ddtrace.bootstrap.preload 1.044 ms (0.43%)
ddtrace.internal.remoteconfig.client 0.552 ms (0.23%)
ddtrace 0.672 ms (0.28%)
ddtrace.internal._unpatched 0.025 ms (0.01%)
json 0.025 ms (0.01%)
json.decoder 0.025 ms (0.01%)
re 0.025 ms (0.01%)
enum 0.025 ms (0.01%)
types 0.025 ms (0.01%)

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