-
Notifications
You must be signed in to change notification settings - Fork 468
chore(telemetry): add process tags #15227
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
|
Bootstrap import analysisComparison of import times between this PR and base. SummaryThe average import time from this PR is: 219 ± 3 ms. The average import time from base is: 227 ± 4 ms. The import time difference between this PR and base is: -8.5 ± 0.2 ms. Import time breakdownThe following import paths have appeared:
|
7e097ee to
256f2de
Compare
This PR implements this [RFC](https://docs.google.com/document/d/1AFdLUuVk70i0bJd5335-RxqsvwAV9ovAqcO2z5mEMbA/edit?pli=1&tab=t.0#heading=h.s9l1lctqlg11) for dynamic instrumentation. Add process_tags to dynamic instrumentation payload ## Testing - Check that process tags are not included if deactivated - Check the right process tags are set in the payload when activated
e338aa1 to
8341969
Compare
This PR implements this RFC for telemetry.
Add process_tags to telemetry configuration
Testing