Repository navigation
Report debugger telemetry metrics in the debugger namespace - #12892
gh-worker-dd-mergequeue-cf854d[bot] merged 1 commit into
Conversation
The debugger team agreed to report all debugger metrics in a `debugger` instrumentation telemetry namespace, shared by every tracer, instead of `live_debugger`. Environment: Datadog workspace Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
There was a problem hiding this comment.
More details
Both debugger metric types receive the shared debugger namespace, and the telemetry pipeline forwards it without restrictions. Release compatibility depends on the intake support described in the PR being deployed.
🤖 Bits Code Review · Commit cbbd732
This comment has been minimized.
This comment has been minimized.
🟢 Java Benchmark SLOs — All performance SLOs passed
PR vs. master results
Commit: Load and DaCapo benchmarks can be triggered manually in the GitLab pipeline. Results will appear in the Benchmarking Platform UI after completion. |
|
/merge |
|
View all feedbacks in Devflow UI.
The expected merge time in
thomas.watson@datadoghq.com cancelled this merge request build |
7634ad8
into
master
Ruby 2.46.0 reports its debugger metrics in the `debugger` namespace (DataDog/dd-trace-rb#6484), and Java's change (DataDog/dd-trace-java#12892) is merged, so enable the test for Ruby from 2.46.0 and for Java from the 1.68.0 snapshots. Both use the weblogs where the line-probe snapshot tests run, since this test installs a line snapshot probe. Environment: Datadog workspace Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Summary
The debugger team agreed that every tracer reports its debugger metrics in a shared
debuggerinstrumentation telemetry namespace instead oflive_debugger.The intake allows the new namespace since ddoghq/dd-go#23524. DataDog/system-tests#8052 adds a cross-tracer test for it.
Evidence
DebuggerMetricCollectorTestassertsmetric.namespace == "live_debugger".After: it asserts
metric.namespace == "debugger". I couldn't run the Gradle build locally, so CI is the first run.Merge Danger
Door: two-way
Blast Radius: internal
Only internal telemetry changes. The intake change (ddoghq/dd-go#23524) is merged; it must be deployed before a release containing this ships, because until then the intake rejects
generate-metricspayloads that containdebuggerseries.Jira ticket: DEBUG-6335
🤖 Generated with Claude Code