Skip to content

Add telemetry metrics for dropped/skipped events - #12319

Open
jpbempel wants to merge 5 commits into
masterfrom
jpbempel/add-debugger-telemetry
Open

Add telemetry metrics for dropped/skipped events#12319
jpbempel wants to merge 5 commits into
masterfrom
jpbempel/add-debugger-telemetry

Conversation

@jpbempel

@jpbempel jpbempel commented Aug 27, 2026

Copy link
Copy Markdown
Member

What Does This Do

when snapshots are dropped (Queue full) or skipped (rate limit or eval timeout) we are sending telemetry metrics with te number of events (snapshots) involved. metrics are tagged by reason

we are removing the old DebuggerMetrics and the skip cause to replace by this new telemetry metrics.
More telemetry metrics will be added later

Motivation

Additional Notes

Contributor Checklist

Jira ticket: DEBUG-5839

when snapshots are dropped (Queue full) or skipped (rate limit or
eval timeout) we are sending telemetry metrics with te number of
events (snapshots) involved. metrics are tagged by reason

we are removing the old DebuggerMetrics and the skip cause to replace
by this new telemetry metrics.
More telemetry metrics will be added later
@jpbempel
jpbempel requested review from a team as code owners August 27, 2026 15:05
@jpbempel
jpbempel requested review from dudikeleti and sarahchen6 and removed request for a team August 27, 2026 15:05
@jpbempel jpbempel added the comp: debugger Dynamic Instrumentation label Aug 27, 2026
@dd-octo-sts

dd-octo-sts Bot commented Aug 27, 2026

Copy link
Copy Markdown
Contributor

Hi! 👋 Thanks for your pull request! 🎉

To help us review it, please make sure to:

  • Add at least one type, and one component or instrumentation label to the pull request

If you need help, please check our contributing guidelines.

@jpbempel jpbempel added type: refactoring type: feature Enhancements and improvements labels Aug 27, 2026

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 560d92d527

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

@datadog-prod-us1-4 datadog-prod-us1-4 Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Datadog Autotest: FAIL

The metric collector can lose event counts during a concurrent reset. Some timeout and disabled-session paths also omit or use an incorrect skip reason.

Open Bits AI session

🤖 Datadog Autotest · Commit 560d92d · What is Autotest? · @DataDog review to ask questions · Any feedback? Reach out in #autotest

@datadog-prod-us1-4

This comment has been minimized.

@pr-commenter

pr-commenter Bot commented Aug 27, 2026

Copy link
Copy Markdown

Debugger benchmarks

Parameters

Baseline Candidate
baseline_or_candidate baseline candidate
ci_job_date 1787908608 1787908934
end_time 2026-08-28T09:18:15 2026-08-28T09:23:42
git_branch master jpbempel/add-debugger-telemetry
git_commit_sha 86c5198 89cdeff
start_time 2026-08-28T09:16:49 2026-08-28T09:22:15
See matching parameters
Baseline Candidate
ci_job_id 1993279825 1993279825
ci_pipeline_id 133865532 133865532
cpu_model Intel(R) Xeon(R) Platinum 8259CL CPU @ 2.50GHz Intel(R) Xeon(R) Platinum 8259CL CPU @ 2.50GHz
git_commit_date 1787907974 1787907974

Summary

Found 0 performance improvements and 0 performance regressions! Performance is the same for 10 metrics, 5 unstable metrics.

See unchanged results
scenario Δ mean agg_http_req_duration_min Δ mean agg_http_req_duration_p50 Δ mean agg_http_req_duration_p75 Δ mean agg_http_req_duration_p99 Δ mean throughput
scenario:noprobe unstable
[-18.075µs; +13.279µs] or [-5.836%; +4.287%]
unstable
[-23.173µs; +29.501µs] or [-6.787%; +8.641%]
unstable
[-31.083µs; +40.996µs] or [-8.742%; +11.530%]
unstable
[-104.554µs; +107.060µs] or [-8.260%; +8.457%]
same
scenario:basic same same same unstable
[-89.846µs; +133.694µs] or [-7.897%; +11.751%]
same
scenario:loop same same same same same
Request duration reports for reports
gantt
    title reports - request duration [CI 0.99] : candidate=None, baseline=None
    dateFormat X
    axisFormat %s
section baseline
noprobe (341.413 µs) : 314, 368
.   : milestone, 341,
basic (322.415 µs) : 315, 330
.   : milestone, 322,
loop (8.096 ms) : 8034, 8158
.   : milestone, 8096,
section candidate
noprobe (344.577 µs) : 322, 368
.   : milestone, 345,
basic (322.759 µs) : 317, 329
.   : milestone, 323,
loop (8.091 ms) : 8028, 8154
.   : milestone, 8091,
Loading
  • baseline results
Scenario Request median duration [CI 0.99]
noprobe 341.413 µs [314.462 µs, 368.365 µs]
basic 322.415 µs [315.062 µs, 329.769 µs]
loop 8.096 ms [8.034 ms, 8.158 ms]
  • candidate results
Scenario Request median duration [CI 0.99]
noprobe 344.577 µs [321.637 µs, 367.518 µs]
basic 322.759 µs [316.814 µs, 328.704 µs]
loop 8.091 ms [8.028 ms, 8.154 ms]

@dd-octo-sts

dd-octo-sts Bot commented Aug 27, 2026

Copy link
Copy Markdown
Contributor

🟢 Java Benchmark SLOs — All performance SLOs passed

Suite Status
Startup 🟢 pass

SLO thresholds are defined here based on automatically generated metrics. A warning is raised when results are within 5% of the threshold.

PR vs. master results
Scenario Candidate master Δ (95% CI of mean)
startup:insecure-bank:iast:Agent 14.05 s 14.00 s [-0.5%; +1.3%] (no difference)
startup:insecure-bank:tracing:Agent 12.91 s 13.03 s [-1.5%; -0.3%] (maybe better)
startup:petclinic:appsec:Agent 17.57 s 17.45 s [-0.0%; +1.4%] (no difference)
startup:petclinic:iast:Agent 17.49 s 17.53 s [-1.1%; +0.6%] (no difference)
startup:petclinic:profiling:Agent 17.45 s 17.48 s [-1.5%; +1.1%] (no difference)
startup:petclinic:sca:Agent 17.49 s 17.29 s [+0.1%; +2.2%] (maybe worse)
startup:petclinic:tracing:Agent 16.59 s 16.64 s [-1.5%; +1.0%] (no difference)

Commit: 96158d3d · CI Pipeline · Benchmarking Platform UI


Load and DaCapo benchmarks can be triggered manually in the GitLab pipeline. Results will appear in the Benchmarking Platform UI after completion.

@sarahchen6 sarahchen6 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks like Jacoco is complaining about test coverage, but otherwise LGTM

@jpbempel
jpbempel enabled auto-merge August 28, 2026 12:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

comp: debugger Dynamic Instrumentation type: feature Enhancements and improvements type: refactoring

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants