[Tracer] Add new process tags by vandonr · Pull Request #7651 · DataDog/dd-trace-dotnet
added 2 commits
October 8, 2025 14:11
lucaspimentel
added
the
area:tracer
label
Oct 24, 2025
lucaspimentel
changed the title
Write process tags in trace chunks
[Tracer] Add new process tags
vandonr added a commit that referenced this pull request
Oct 30, 2025## Summary of changes addressing late comments on #7651 ## Reason for change ## Implementation details ## Test coverage ## Other details <!-- Fixes #{issue} --> <!-- ⚠️ Note: Where possible, please obtain 2 approvals prior to merging. Unless CODEOWNERS specifies otherwise, for external teams it is typically best to have one review from a team member, and one review from apm-dotnet. Trivial changes do not require 2 reviews. MergeQueue is NOT enabled in this repository. If you have write access to the repo, the PR has 1-2 approvals (see above), and all of the required checks have passed, you can use the Squash and Merge button to merge the PR. If you don't have write access, or you need help, reach out in the #apm-dotnet channel in Slack. -->
vandonr added a commit that referenced this pull request
Nov 17, 2025## Summary of changes - add a hash of process tags to the base hash of pathway checkpoint hashes - write the process tags in DSM payloads(so that the backend can resolve the right service name) Since this makes us query the process tags during the tracer initialization, I removed the Lazy mechanism that I had put in place previously, because there would be nothing lazy about it anymore. This means that values that are available only later in the app life (like IIS entrypoint for instance) are not accessible. still controlled by the same feature flag as in #7651 This is the dotnet equivalent of DataDog/dd-trace-java#8836 ## Reason for change [AIDM-195] ## Implementation details ## Test coverage ## Other details <!-- Fixes #{issue} --> <!-- ⚠️ Note: Where possible, please obtain 2 approvals prior to merging. Unless CODEOWNERS specifies otherwise, for external teams it is typically best to have one review from a team member, and one review from apm-dotnet. Trivial changes do not require 2 reviews. MergeQueue is NOT enabled in this repository. If you have write access to the repo, the PR has 1-2 approvals (see above), and all of the required checks have passed, you can use the Squash and Merge button to merge the PR. If you don't have write access, or you need help, reach out in the #apm-dotnet channel in Slack. --> [AIDM-195]: https://datadoghq.atlassian.net/browse/AIDM-195?atlOrigin=eyJpIjoiNWRkNTljNzYxNjVmNDY3MDlhMDU5Y2ZhYzA5YTRkZjUiLCJwIjoiZ2l0aHViLWNvbS1KU1cifQ
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters