-
Notifications
You must be signed in to change notification settings - Fork 134
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
RUM-1836 feat(otel-tracer): send telemetry for Tracer API usage #1704
RUM-1836 feat(otel-tracer): send telemetry for Tracer API usage #1704
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Shouldn't we track metric
instead of debug
in both cases?
When I was talking with @ncreated, recommendation was to use debug but happy to change whatever makes sense here. End goal is to have some tracking mechanism. |
It doesn't seem to be a |
1198e51
to
f0cab01
Compare
// Generated from https://github.com/DataDog/rum-events-format/tree/389581be98dcf8efbfcfe7bffaa32d53f960fb6f | ||
// Generated from https://github.com/DataDog/rum-events-format/tree/b41d863ee439a22cef58fd2732e99b2cb8e4d89d |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I can't find b41d863 in rum-events-format
repo 🤔 - we need a PR-approved change merged to rum-events-format/master
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There are other unrelated issue due to branch being back.
f0cab01
to
ec37f70
Compare
3c553a2
to
4ea1bb6
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🎯
What?
We want to track how many customers are using the OTel vs OT APIs
Schema update: DataDog/rum-events-format#190
How?
Send config telemetry events when tracer is get from a public API.
The initial plan to send the telemetry log when tracer is initialized which gives accurate data but for consistency reasons, we decided to use configuration telemetry. We may miss data for cases if the tracer is not initialized in first 5 seconds of the creation of configuration telemetry.
Review checklist
Custom CI job configuration (optional)
tools/