Repository navigation
Conversation
…space The debugger team agreed that every tracer reports its debugger metrics in a shared `debugger` instrumentation telemetry namespace, instead of `live_debugger` or `dynamic_instrumentation`. Add Test_Debugger_Telemetry_Metrics: it hits a snapshot line probe in a quick burst so the per-probe rate limit skips events, then asserts that `events.skipped` tagged `reason:rateLimitProbe` is reported in the `debugger` namespace and that no debugger metric is sent in a legacy namespace. The test is disabled for every tracer until each one ships the namespace change. Environment: Datadog workspace Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Contributor
|
|
This was referenced Oct 15, 2026
Resolve the conflict in manifests/python.yml: keep main's new `aiohttp` weblog declarations for the debugger telemetry tests, and keep the `Test_Debugger_Telemetry_Metrics` entry after them. Environment: Datadog workspace Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
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>
DataDog/dd-trace-js#10757 is on the v6.22.0 and v5.133.0 release proposals, so enable the test from those versions on every weblog where the line-probe snapshot tests run (all but nextjs). Add a ref_6_22_0 anchor for the version range. Environment: Datadog workspace Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
🎉 All green!🧪 All tests passed 🔗 Commit SHA: 9b08d08 | Docs | View more details | Give us feedback! |
DataDog/dd-trace-dotnet#9423 is merged and ships in 3.57.0 (3.56.0 was released the day before), so enable the test from the 3.57.0 dev builds, on every weblog like the .NET line-probe snapshot tests. Environment: Datadog workspace Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
The first CI run showed Java emitting a snapshot for all 10 hits, so it skipped nothing and reported no events.skipped metric. Java and .NET rate limit snapshot probes with an adaptive sampler that starts with an unused budget of 16 extra snapshots on top of the 1 per second, so a burst of 10 never reaches the limit. A burst of 50 does, with margin for the budget that refills during the burst. Environment: Datadog workspace Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This branch has not been deployed
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
The debugger team agreed that every tracer reports its debugger metrics in a shared
debuggerinstrumentation telemetry namespace, instead oflive_debuggerordynamic_instrumentation(DEBUG-6335). This adds a cross-tracer test for it in theDEBUGGER_TELEMETRYscenario.The test only relies on
reason:rateLimitProbe, the one guardrail every tracer already emits. Java doesn't sendevent_type.Manifests enable it where the change has shipped or reached dev builds, using the weblogs where the line-probe snapshot tests run:
v2.46.0(rails72,rails80,uds-rails)v1.68.0-SNAPSHOT(Spring Boot weblogs)>=6.22.0 || ^5.133.0(all weblogs butnextjs)v3.57.0-dev(all weblogs)events.skippedyetmissing_featuremissing_featureThe intake allows the namespace in ddoghq/dd-go#23524.
Evidence
generate-metricsandsketchespayloads, covering payload-level and per-series namespaces../format.shpasses.Merge Danger
Door: two-way
Blast Radius: small
The new test runs for Ruby, Java, Node.js and .NET. It has never run end to end before (see Evidence), so their CI jobs are its first real run.
🤖 Generated with Claude Code