-
Notifications
You must be signed in to change notification settings - Fork 293
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
Enhance log probes to honor debug session tags #8215
Conversation
Hi! 👋 Thanks for your pull request! 🎉 To help us review it, please make sure to:
If you need help, please check our contributing guidelines. |
Debugger benchmarksParameters
See matching parameters
SummaryFound 0 performance improvements and 2 performance regressions! Performance is the same for 7 metrics, 6 unstable metrics.
See unchanged results
Request duration reports for reportsgantt
title reports - request duration [CI 0.99] : candidate=None, baseline=None
dateFormat X
axisFormat %s
section baseline
noprobe (307.093 µs) : 285, 329
. : milestone, 307,
basic (312.278 µs) : 303, 321
. : milestone, 312,
loop (9.998 ms) : 9977, 10019
. : milestone, 9998,
section candidate
noprobe (320.288 µs) : 268, 373
. : milestone, 320,
basic (314.915 µs) : 304, 326
. : milestone, 315,
loop (10.142 ms) : 10110, 10175
. : milestone, 10142,
|
BenchmarksStartupParameters
See matching parameters
SummaryFound 1 performance improvements and 0 performance regressions! Performance is the same for 57 metrics, 5 unstable metrics.
Startup time reports for petclinicgantt
title petclinic - global startup overhead: candidate=1.46.0-SNAPSHOT~db4aefa14b, baseline=1.46.0-SNAPSHOT~fbb36f9b5b
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.054 s) : 0, 1053610
Total [baseline] (10.481 s) : 0, 10480905
Agent [candidate] (1.061 s) : 0, 1060892
Total [candidate] (10.572 s) : 0, 10572292
section appsec
Agent [baseline] (1.193 s) : 0, 1192701
Total [baseline] (10.698 s) : 0, 10697641
Agent [candidate] (1.192 s) : 0, 1192482
Total [candidate] (10.747 s) : 0, 10747405
section iast
Agent [baseline] (1.203 s) : 0, 1203055
Total [baseline] (11.099 s) : 0, 11099351
Agent [candidate] (1.199 s) : 0, 1198807
Total [candidate] (11.005 s) : 0, 11004994
section profiling
Agent [baseline] (1.253 s) : 0, 1253192
Total [baseline] (10.795 s) : 0, 10795119
Agent [candidate] (1.253 s) : 0, 1253133
Total [candidate] (10.847 s) : 0, 10846681
gantt
title petclinic - break down per module: candidate=1.46.0-SNAPSHOT~db4aefa14b, baseline=1.46.0-SNAPSHOT~fbb36f9b5b
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (713.072 ms) : 0, 713072
BytebuddyAgent [candidate] (717.49 ms) : 0, 717490
GlobalTracer [baseline] (255.796 ms) : 0, 255796
GlobalTracer [candidate] (257.177 ms) : 0, 257177
AppSec [baseline] (56.125 ms) : 0, 56125
AppSec [candidate] (55.697 ms) : 0, 55697
Remote Config [baseline] (738.969 µs) : 0, 739
Remote Config [candidate] (734.879 µs) : 0, 735
Telemetry [baseline] (12.875 ms) : 0, 12875
Telemetry [candidate] (14.651 ms) : 0, 14651
section appsec
BytebuddyAgent [baseline] (733.922 ms) : 0, 733922
BytebuddyAgent [candidate] (733.364 ms) : 0, 733364
GlobalTracer [baseline] (253.644 ms) : 0, 253644
GlobalTracer [candidate] (253.949 ms) : 0, 253949
AppSec [baseline] (171.487 ms) : 0, 171487
AppSec [candidate] (171.503 ms) : 0, 171503
IAST [baseline] (19.429 ms) : 0, 19429
IAST [candidate] (19.454 ms) : 0, 19454
Remote Config [baseline] (668.397 µs) : 0, 668
Remote Config [candidate] (666.839 µs) : 0, 667
Telemetry [baseline] (8.217 ms) : 0, 8217
Telemetry [candidate] (8.209 ms) : 0, 8209
section iast
BytebuddyAgent [baseline] (848.08 ms) : 0, 848080
BytebuddyAgent [candidate] (843.62 ms) : 0, 843620
GlobalTracer [baseline] (249.491 ms) : 0, 249491
GlobalTracer [candidate] (249.557 ms) : 0, 249557
AppSec [baseline] (58.708 ms) : 0, 58708
AppSec [candidate] (58.541 ms) : 0, 58541
IAST [baseline] (21.867 ms) : 0, 21867
IAST [candidate] (21.989 ms) : 0, 21989
Remote Config [baseline] (674.54 µs) : 0, 675
Remote Config [candidate] (682.634 µs) : 0, 683
Telemetry [baseline] (8.95 ms) : 0, 8950
Telemetry [candidate] (8.95 ms) : 0, 8950
section profiling
BytebuddyAgent [baseline] (702.889 ms) : 0, 702889
BytebuddyAgent [candidate] (702.262 ms) : 0, 702262
GlobalTracer [baseline] (348.862 ms) : 0, 348862
GlobalTracer [candidate] (350.325 ms) : 0, 350325
AppSec [baseline] (54.931 ms) : 0, 54931
AppSec [candidate] (53.442 ms) : 0, 53442
Remote Config [baseline] (711.96 µs) : 0, 712
Remote Config [candidate] (654.846 µs) : 0, 655
Telemetry [baseline] (8.815 ms) : 0, 8815
Telemetry [candidate] (8.825 ms) : 0, 8825
ProfilingAgent [baseline] (95.065 ms) : 0, 95065
ProfilingAgent [candidate] (95.543 ms) : 0, 95543
Profiling [baseline] (95.088 ms) : 0, 95088
Profiling [candidate] (95.567 ms) : 0, 95567
Startup time reports for insecure-bankgantt
title insecure-bank - global startup overhead: candidate=1.46.0-SNAPSHOT~db4aefa14b, baseline=1.46.0-SNAPSHOT~fbb36f9b5b
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.066 s) : 0, 1065521
Total [baseline] (8.686 s) : 0, 8686464
Agent [candidate] (1.054 s) : 0, 1054218
Total [candidate] (8.624 s) : 0, 8623779
section iast
Agent [baseline] (1.188 s) : 0, 1188404
Total [baseline] (9.21 s) : 0, 9209527
Agent [candidate] (1.182 s) : 0, 1182102
Total [candidate] (9.199 s) : 0, 9199438
section iast_HARDCODED_SECRET_DISABLED
Agent [baseline] (1.185 s) : 0, 1184805
Total [baseline] (9.167 s) : 0, 9166692
Agent [candidate] (1.196 s) : 0, 1195872
Total [candidate] (9.28 s) : 0, 9279565
section iast_TELEMETRY_OFF
Agent [baseline] (1.179 s) : 0, 1178550
Total [baseline] (9.219 s) : 0, 9219169
Agent [candidate] (1.177 s) : 0, 1176703
Total [candidate] (9.218 s) : 0, 9218021
gantt
title insecure-bank - break down per module: candidate=1.46.0-SNAPSHOT~db4aefa14b, baseline=1.46.0-SNAPSHOT~fbb36f9b5b
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (723.228 ms) : 0, 723228
BytebuddyAgent [candidate] (712.746 ms) : 0, 712746
GlobalTracer [baseline] (259.275 ms) : 0, 259275
GlobalTracer [candidate] (255.723 ms) : 0, 255723
AppSec [baseline] (56.859 ms) : 0, 56859
AppSec [candidate] (55.793 ms) : 0, 55793
Remote Config [baseline] (749.022 µs) : 0, 749
Remote Config [candidate] (725.336 µs) : 0, 725
Telemetry [baseline] (10.194 ms) : 0, 10194
Telemetry [candidate] (14.246 ms) : 0, 14246
section iast
BytebuddyAgent [baseline] (836.041 ms) : 0, 836041
BytebuddyAgent [candidate] (831.156 ms) : 0, 831156
GlobalTracer [baseline] (247.31 ms) : 0, 247310
GlobalTracer [candidate] (247.023 ms) : 0, 247023
AppSec [baseline] (58.496 ms) : 0, 58496
AppSec [candidate] (57.961 ms) : 0, 57961
IAST [baseline] (21.806 ms) : 0, 21806
IAST [candidate] (21.552 ms) : 0, 21552
Remote Config [baseline] (697.192 µs) : 0, 697
Remote Config [candidate] (683.646 µs) : 0, 684
Telemetry [baseline] (8.961 ms) : 0, 8961
Telemetry [candidate] (8.761 ms) : 0, 8761
section iast_HARDCODED_SECRET_DISABLED
BytebuddyAgent [baseline] (832.53 ms) : 0, 832530
BytebuddyAgent [candidate] (840.687 ms) : 0, 840687
GlobalTracer [baseline] (247.257 ms) : 0, 247257
GlobalTracer [candidate] (248.875 ms) : 0, 248875
AppSec [baseline] (58.638 ms) : 0, 58638
AppSec [candidate] (58.936 ms) : 0, 58936
IAST [baseline] (21.656 ms) : 0, 21656
IAST [candidate] (22.275 ms) : 0, 22275
Remote Config [baseline] (674.818 µs) : 0, 675
Remote Config [candidate] (705.673 µs) : 0, 706
Telemetry [baseline] (9.005 ms) : 0, 9005
Telemetry [candidate] (9.175 ms) : 0, 9175
section iast_TELEMETRY_OFF
BytebuddyAgent [baseline] (828.408 ms) : 0, 828408
BytebuddyAgent [candidate] (827.345 ms) : 0, 827345
GlobalTracer [baseline] (246.996 ms) : 0, 246996
GlobalTracer [candidate] (246.291 ms) : 0, 246291
AppSec [baseline] (57.992 ms) : 0, 57992
AppSec [candidate] (57.9 ms) : 0, 57900
IAST [baseline] (20.794 ms) : 0, 20794
IAST [candidate] (20.875 ms) : 0, 20875
Remote Config [baseline] (662.324 µs) : 0, 662
Remote Config [candidate] (659.2 µs) : 0, 659
Telemetry [baseline] (8.679 ms) : 0, 8679
Telemetry [candidate] (8.602 ms) : 0, 8602
LoadParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 12 metrics, 16 unstable metrics. Request duration reports for petclinicgantt
title petclinic - request duration [CI 0.99] : candidate=1.46.0-SNAPSHOT~db4aefa14b, baseline=1.46.0-SNAPSHOT~fbb36f9b5b
dateFormat X
axisFormat %s
section baseline
no_agent (1.366 ms) : 1346, 1385
. : milestone, 1366,
appsec (1.748 ms) : 1724, 1772
. : milestone, 1748,
appsec_no_iast (1.754 ms) : 1731, 1776
. : milestone, 1754,
iast (1.506 ms) : 1481, 1530
. : milestone, 1506,
profiling (1.51 ms) : 1486, 1534
. : milestone, 1510,
tracing (1.489 ms) : 1464, 1514
. : milestone, 1489,
section candidate
no_agent (1.364 ms) : 1344, 1384
. : milestone, 1364,
appsec (1.753 ms) : 1729, 1776
. : milestone, 1753,
appsec_no_iast (1.776 ms) : 1752, 1800
. : milestone, 1776,
iast (1.518 ms) : 1494, 1542
. : milestone, 1518,
profiling (1.528 ms) : 1504, 1552
. : milestone, 1528,
tracing (1.479 ms) : 1454, 1503
. : milestone, 1479,
Request duration reports for insecure-bankgantt
title insecure-bank - request duration [CI 0.99] : candidate=1.46.0-SNAPSHOT~db4aefa14b, baseline=1.46.0-SNAPSHOT~fbb36f9b5b
dateFormat X
axisFormat %s
section baseline
no_agent (385.578 µs) : 366, 406
. : milestone, 386,
iast (509.385 µs) : 488, 531
. : milestone, 509,
iast_FULL (740.557 µs) : 719, 762
. : milestone, 741,
iast_GLOBAL (563.735 µs) : 541, 587
. : milestone, 564,
iast_HARDCODED_SECRET_DISABLED (508.006 µs) : 486, 530
. : milestone, 508,
iast_INACTIVE (467.199 µs) : 445, 489
. : milestone, 467,
iast_TELEMETRY_OFF (510.89 µs) : 489, 533
. : milestone, 511,
tracing (449.996 µs) : 429, 471
. : milestone, 450,
section candidate
no_agent (379.352 µs) : 359, 399
. : milestone, 379,
iast (507.138 µs) : 486, 529
. : milestone, 507,
iast_FULL (740.636 µs) : 719, 762
. : milestone, 741,
iast_GLOBAL (562.178 µs) : 539, 585
. : milestone, 562,
iast_HARDCODED_SECRET_DISABLED (513.018 µs) : 491, 535
. : milestone, 513,
iast_INACTIVE (466.211 µs) : 444, 489
. : milestone, 466,
iast_TELEMETRY_OFF (496.926 µs) : 475, 519
. : milestone, 497,
tracing (447.913 µs) : 428, 468
. : milestone, 448,
DacapoParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 12 metrics, 0 unstable metrics. Execution time for tomcatgantt
title tomcat - execution time [CI 0.99] : candidate=1.46.0-SNAPSHOT~db4aefa14b, baseline=1.46.0-SNAPSHOT~fbb36f9b5b
dateFormat X
axisFormat %s
section baseline
no_agent (1.465 ms) : 1454, 1477
. : milestone, 1465,
appsec (2.36 ms) : 2317, 2403
. : milestone, 2360,
iast (2.107 ms) : 2052, 2162
. : milestone, 2107,
iast_GLOBAL (2.152 ms) : 2096, 2207
. : milestone, 2152,
profiling (1.972 ms) : 1928, 2016
. : milestone, 1972,
tracing (1.943 ms) : 1901, 1985
. : milestone, 1943,
section candidate
no_agent (1.468 ms) : 1457, 1480
. : milestone, 1468,
appsec (2.346 ms) : 2303, 2389
. : milestone, 2346,
iast (2.102 ms) : 2048, 2157
. : milestone, 2102,
iast_GLOBAL (2.151 ms) : 2096, 2206
. : milestone, 2151,
profiling (1.949 ms) : 1906, 1993
. : milestone, 1949,
tracing (1.941 ms) : 1899, 1983
. : milestone, 1941,
Execution time for biojavagantt
title biojava - execution time [CI 0.99] : candidate=1.46.0-SNAPSHOT~db4aefa14b, baseline=1.46.0-SNAPSHOT~fbb36f9b5b
dateFormat X
axisFormat %s
section baseline
no_agent (14.644 s) : 14644000, 14644000
. : milestone, 14644000,
appsec (15.104 s) : 15104000, 15104000
. : milestone, 15104000,
iast (18.505 s) : 18505000, 18505000
. : milestone, 18505000,
iast_GLOBAL (17.94 s) : 17940000, 17940000
. : milestone, 17940000,
profiling (15.007 s) : 15007000, 15007000
. : milestone, 15007000,
tracing (14.956 s) : 14956000, 14956000
. : milestone, 14956000,
section candidate
no_agent (15.587 s) : 15587000, 15587000
. : milestone, 15587000,
appsec (14.901 s) : 14901000, 14901000
. : milestone, 14901000,
iast (18.694 s) : 18694000, 18694000
. : milestone, 18694000,
iast_GLOBAL (17.867 s) : 17867000, 17867000
. : milestone, 17867000,
profiling (14.968 s) : 14968000, 14968000
. : milestone, 14968000,
tracing (15.032 s) : 15032000, 15032000
. : milestone, 15032000,
|
dd-java-agent/agent-debugger/src/main/java/com/datadog/debugger/probe/ProbeDefinition.java
Outdated
Show resolved
Hide resolved
dd-java-agent/agent-debugger/src/main/java/com/datadog/debugger/probe/LogProbe.java
Outdated
Show resolved
Hide resolved
dd-java-agent/agent-debugger/src/main/java/com/datadog/debugger/probe/ProbeDefinition.java
Outdated
Show resolved
Hide resolved
dd-java-agent/agent-debugger/src/main/java/com/datadog/debugger/sink/DebuggerSink.java
Show resolved
Hide resolved
dd-java-agent/agent-debugger/src/main/java/com/datadog/debugger/probe/ProbeDefinition.java
Outdated
Show resolved
Hide resolved
PR clean up
PR clean up
What Does This Do
Enhances log probes to honor session id tags for live debugger support.
Motivation
Additional Notes
Contributor Checklist
type:
and (comp:
orinst:
) labels in addition to any usefull labelsclose
,fix
or any linking keywords when referencing an issue.Use
solves
instead, and assign the PR milestone to the issueJira ticket: DEBUG-2955