-
Notifications
You must be signed in to change notification settings - Fork 318
Don't instrument trigger probes if the distributed debugger is disabled #8458
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
Conversation
Debugger benchmarksParameters
See matching parameters
SummaryFound 0 performance improvements and 4 performance regressions! Performance is the same for 5 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 (317.406 µs) : 278, 357
. : milestone, 317,
basic (311.996 µs) : 302, 322
. : milestone, 312,
loop (10.129 ms) : 10080, 10177
. : milestone, 10129,
section candidate
noprobe (316.75 µs) : 273, 360
. : milestone, 317,
basic (304.098 µs) : 294, 314
. : milestone, 304,
loop (10.482 ms) : 10415, 10549
. : milestone, 10482,
|
BenchmarksStartupParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 56 metrics, 7 unstable metrics. Startup time reports for petclinicgantt
title petclinic - global startup overhead: candidate=1.47.0-SNAPSHOT~88a893c3ef, baseline=1.47.0-SNAPSHOT~19d8832a9d
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.047 s) : 0, 1047076
Total [baseline] (10.467 s) : 0, 10466807
Agent [candidate] (1.039 s) : 0, 1038553
Total [candidate] (10.504 s) : 0, 10503548
section appsec
Agent [baseline] (1.184 s) : 0, 1184471
Total [baseline] (10.719 s) : 0, 10719432
Agent [candidate] (1.181 s) : 0, 1180501
Total [candidate] (10.726 s) : 0, 10726021
section iast
Agent [baseline] (1.171 s) : 0, 1171078
Total [baseline] (11.025 s) : 0, 11024947
Agent [candidate] (1.175 s) : 0, 1175248
Total [candidate] (11.014 s) : 0, 11013963
section profiling
Agent [baseline] (1.262 s) : 0, 1261641
Total [baseline] (10.83 s) : 0, 10830379
Agent [candidate] (1.268 s) : 0, 1267750
Total [candidate] (10.921 s) : 0, 10920855
gantt
title petclinic - break down per module: candidate=1.47.0-SNAPSHOT~88a893c3ef, baseline=1.47.0-SNAPSHOT~19d8832a9d
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (722.179 ms) : 0, 722179
BytebuddyAgent [candidate] (716.507 ms) : 0, 716507
GlobalTracer [baseline] (241.102 ms) : 0, 241102
GlobalTracer [candidate] (239.166 ms) : 0, 239166
AppSec [baseline] (56.272 ms) : 0, 56272
AppSec [candidate] (55.569 ms) : 0, 55569
Remote Config [baseline] (700.476 µs) : 0, 700
Remote Config [candidate] (712.849 µs) : 0, 713
Telemetry [baseline] (11.629 ms) : 0, 11629
Telemetry [candidate] (11.611 ms) : 0, 11611
section appsec
BytebuddyAgent [baseline] (735.993 ms) : 0, 735993
BytebuddyAgent [candidate] (733.549 ms) : 0, 733549
GlobalTracer [baseline] (236.196 ms) : 0, 236196
GlobalTracer [candidate] (235.966 ms) : 0, 235966
AppSec [baseline] (177.141 ms) : 0, 177141
AppSec [candidate] (175.973 ms) : 0, 175973
Remote Config [baseline] (661.843 µs) : 0, 662
Remote Config [candidate] (658.507 µs) : 0, 659
Telemetry [baseline] (8.599 ms) : 0, 8599
Telemetry [candidate] (8.642 ms) : 0, 8642
IAST [baseline] (21.83 ms) : 0, 21830
IAST [candidate] (21.549 ms) : 0, 21549
section iast
BytebuddyAgent [baseline] (837.092 ms) : 0, 837092
BytebuddyAgent [candidate] (840.087 ms) : 0, 840087
GlobalTracer [baseline] (230.153 ms) : 0, 230153
GlobalTracer [candidate] (230.743 ms) : 0, 230743
AppSec [baseline] (57.008 ms) : 0, 57008
AppSec [candidate] (56.967 ms) : 0, 56967
Remote Config [baseline] (591.56 µs) : 0, 592
Remote Config [candidate] (606.183 µs) : 0, 606
Telemetry [baseline] (8.637 ms) : 0, 8637
Telemetry [candidate] (8.731 ms) : 0, 8731
IAST [baseline] (22.658 ms) : 0, 22658
IAST [candidate] (23.036 ms) : 0, 23036
section profiling
BytebuddyAgent [baseline] (709.906 ms) : 0, 709906
BytebuddyAgent [candidate] (714.238 ms) : 0, 714238
GlobalTracer [baseline] (350.006 ms) : 0, 350006
GlobalTracer [candidate] (351.961 ms) : 0, 351961
AppSec [baseline] (55.381 ms) : 0, 55381
AppSec [candidate] (54.687 ms) : 0, 54687
Remote Config [baseline] (678.1 µs) : 0, 678
Remote Config [candidate] (685.622 µs) : 0, 686
Telemetry [baseline] (8.977 ms) : 0, 8977
Telemetry [candidate] (8.996 ms) : 0, 8996
ProfilingAgent [baseline] (96.444 ms) : 0, 96444
ProfilingAgent [candidate] (96.633 ms) : 0, 96633
Profiling [baseline] (96.467 ms) : 0, 96467
Profiling [candidate] (96.657 ms) : 0, 96657
Startup time reports for insecure-bankgantt
title insecure-bank - global startup overhead: candidate=1.47.0-SNAPSHOT~88a893c3ef, baseline=1.47.0-SNAPSHOT~19d8832a9d
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.038 s) : 0, 1038425
Total [baseline] (8.649 s) : 0, 8649029
Agent [candidate] (1.038 s) : 0, 1038370
Total [candidate] (8.66 s) : 0, 8660342
section iast
Agent [baseline] (1.17 s) : 0, 1169945
Total [baseline] (9.265 s) : 0, 9264854
Agent [candidate] (1.173 s) : 0, 1172937
Total [candidate] (9.234 s) : 0, 9234325
section iast_HARDCODED_SECRET_DISABLED
Agent [baseline] (1.172 s) : 0, 1172396
Total [baseline] (9.218 s) : 0, 9217628
Agent [candidate] (1.176 s) : 0, 1176224
Total [candidate] (9.246 s) : 0, 9245610
section iast_TELEMETRY_OFF
Agent [baseline] (1.167 s) : 0, 1166851
Total [baseline] (9.239 s) : 0, 9238651
Agent [candidate] (1.174 s) : 0, 1174360
Total [candidate] (9.25 s) : 0, 9249739
gantt
title insecure-bank - break down per module: candidate=1.47.0-SNAPSHOT~88a893c3ef, baseline=1.47.0-SNAPSHOT~19d8832a9d
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (716.185 ms) : 0, 716185
BytebuddyAgent [candidate] (714.974 ms) : 0, 714974
GlobalTracer [baseline] (238.936 ms) : 0, 238936
GlobalTracer [candidate] (238.747 ms) : 0, 238747
AppSec [baseline] (55.475 ms) : 0, 55475
AppSec [candidate] (55.297 ms) : 0, 55297
Remote Config [baseline] (693.111 µs) : 0, 693
Remote Config [candidate] (694.997 µs) : 0, 695
Telemetry [baseline] (12.203 ms) : 0, 12203
Telemetry [candidate] (13.684 ms) : 0, 13684
section iast
BytebuddyAgent [baseline] (836.275 ms) : 0, 836275
BytebuddyAgent [candidate] (838.665 ms) : 0, 838665
GlobalTracer [baseline] (229.824 ms) : 0, 229824
GlobalTracer [candidate] (229.912 ms) : 0, 229912
IAST [baseline] (22.808 ms) : 0, 22808
IAST [candidate] (22.928 ms) : 0, 22928
AppSec [baseline] (56.814 ms) : 0, 56814
AppSec [candidate] (57.009 ms) : 0, 57009
Remote Config [baseline] (616.042 µs) : 0, 616
Remote Config [candidate] (613.014 µs) : 0, 613
Telemetry [baseline] (8.663 ms) : 0, 8663
Telemetry [candidate] (8.744 ms) : 0, 8744
section iast_HARDCODED_SECRET_DISABLED
BytebuddyAgent [baseline] (837.571 ms) : 0, 837571
BytebuddyAgent [candidate] (840.517 ms) : 0, 840517
GlobalTracer [baseline] (230.547 ms) : 0, 230547
GlobalTracer [candidate] (230.83 ms) : 0, 230830
IAST [baseline] (23.44 ms) : 0, 23440
IAST [candidate] (23.197 ms) : 0, 23197
AppSec [baseline] (56.529 ms) : 0, 56529
AppSec [candidate] (57.148 ms) : 0, 57148
Remote Config [baseline] (606.179 µs) : 0, 606
Remote Config [candidate] (620.002 µs) : 0, 620
Telemetry [baseline] (8.704 ms) : 0, 8704
Telemetry [candidate] (8.875 ms) : 0, 8875
section iast_TELEMETRY_OFF
BytebuddyAgent [baseline] (833.341 ms) : 0, 833341
BytebuddyAgent [candidate] (838.923 ms) : 0, 838923
GlobalTracer [baseline] (229.577 ms) : 0, 229577
GlobalTracer [candidate] (231.191 ms) : 0, 231191
IAST [baseline] (25.609 ms) : 0, 25609
IAST [candidate] (26.6 ms) : 0, 26600
AppSec [baseline] (54.14 ms) : 0, 54140
AppSec [candidate] (53.262 ms) : 0, 53262
Remote Config [baseline] (617.946 µs) : 0, 618
Remote Config [candidate] (614.105 µs) : 0, 614
Telemetry [baseline] (8.607 ms) : 0, 8607
Telemetry [candidate] (8.675 ms) : 0, 8675
LoadParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 13 metrics, 17 unstable metrics. Request duration reports for insecure-bankgantt
title insecure-bank - request duration [CI 0.99] : candidate=1.47.0-SNAPSHOT~88a893c3ef, baseline=1.47.0-SNAPSHOT~19d8832a9d
dateFormat X
axisFormat %s
section baseline
no_agent (380.187 µs) : 359, 401
. : milestone, 380,
iast (505.411 µs) : 483, 528
. : milestone, 505,
iast_FULL (728.869 µs) : 707, 751
. : milestone, 729,
iast_GLOBAL (559.451 µs) : 536, 583
. : milestone, 559,
iast_HARDCODED_SECRET_DISABLED (510.37 µs) : 488, 532
. : milestone, 510,
iast_INACTIVE (464.059 µs) : 443, 485
. : milestone, 464,
iast_TELEMETRY_OFF (495.627 µs) : 473, 519
. : milestone, 496,
tracing (455.452 µs) : 434, 477
. : milestone, 455,
section candidate
no_agent (380.648 µs) : 360, 401
. : milestone, 381,
iast (511.136 µs) : 488, 534
. : milestone, 511,
iast_FULL (720.491 µs) : 699, 742
. : milestone, 720,
iast_GLOBAL (554.516 µs) : 532, 577
. : milestone, 555,
iast_HARDCODED_SECRET_DISABLED (509.012 µs) : 487, 531
. : milestone, 509,
iast_INACTIVE (458.478 µs) : 437, 480
. : milestone, 458,
iast_TELEMETRY_OFF (493.062 µs) : 470, 516
. : milestone, 493,
tracing (454.521 µs) : 433, 476
. : milestone, 455,
Request duration reports for petclinicgantt
title petclinic - request duration [CI 0.99] : candidate=1.47.0-SNAPSHOT~88a893c3ef, baseline=1.47.0-SNAPSHOT~19d8832a9d
dateFormat X
axisFormat %s
section baseline
no_agent (1.358 ms) : 1339, 1378
. : milestone, 1358,
appsec (1.733 ms) : 1709, 1756
. : milestone, 1733,
appsec_no_iast (1.723 ms) : 1698, 1749
. : milestone, 1723,
code_origins (1.671 ms) : 1638, 1703
. : milestone, 1671,
iast (1.514 ms) : 1490, 1539
. : milestone, 1514,
profiling (1.549 ms) : 1523, 1574
. : milestone, 1549,
tracing (1.489 ms) : 1465, 1513
. : milestone, 1489,
section candidate
no_agent (1.349 ms) : 1330, 1368
. : milestone, 1349,
appsec (1.75 ms) : 1726, 1774
. : milestone, 1750,
appsec_no_iast (1.776 ms) : 1751, 1802
. : milestone, 1776,
code_origins (1.69 ms) : 1657, 1723
. : milestone, 1690,
iast (1.511 ms) : 1487, 1535
. : milestone, 1511,
profiling (1.545 ms) : 1520, 1569
. : milestone, 1545,
tracing (1.477 ms) : 1452, 1502
. : milestone, 1477,
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.47.0-SNAPSHOT~88a893c3ef, baseline=1.47.0-SNAPSHOT~19d8832a9d
dateFormat X
axisFormat %s
section baseline
no_agent (1.475 ms) : 1463, 1486
. : milestone, 1475,
appsec (2.338 ms) : 2295, 2382
. : milestone, 2338,
iast (2.119 ms) : 2064, 2174
. : milestone, 2119,
iast_GLOBAL (2.163 ms) : 2108, 2218
. : milestone, 2163,
profiling (1.989 ms) : 1944, 2034
. : milestone, 1989,
tracing (1.959 ms) : 1916, 2002
. : milestone, 1959,
section candidate
no_agent (1.472 ms) : 1461, 1484
. : milestone, 1472,
appsec (2.366 ms) : 2322, 2409
. : milestone, 2366,
iast (2.122 ms) : 2066, 2177
. : milestone, 2122,
iast_GLOBAL (2.173 ms) : 2117, 2229
. : milestone, 2173,
profiling (1.971 ms) : 1927, 2015
. : milestone, 1971,
tracing (1.953 ms) : 1911, 1995
. : milestone, 1953,
Execution time for biojavagantt
title biojava - execution time [CI 0.99] : candidate=1.47.0-SNAPSHOT~88a893c3ef, baseline=1.47.0-SNAPSHOT~19d8832a9d
dateFormat X
axisFormat %s
section baseline
no_agent (15.03 s) : 15030000, 15030000
. : milestone, 15030000,
appsec (15.032 s) : 15032000, 15032000
. : milestone, 15032000,
iast (18.937 s) : 18937000, 18937000
. : milestone, 18937000,
iast_GLOBAL (18.211 s) : 18211000, 18211000
. : milestone, 18211000,
profiling (14.909 s) : 14909000, 14909000
. : milestone, 14909000,
tracing (14.95 s) : 14950000, 14950000
. : milestone, 14950000,
section candidate
no_agent (14.911 s) : 14911000, 14911000
. : milestone, 14911000,
appsec (14.778 s) : 14778000, 14778000
. : milestone, 14778000,
iast (18.66 s) : 18660000, 18660000
. : milestone, 18660000,
iast_GLOBAL (17.652 s) : 17652000, 17652000
. : milestone, 17652000,
profiling (15.278 s) : 15278000, 15278000
. : milestone, 15278000,
tracing (14.946 s) : 14946000, 14946000
. : milestone, 14946000,
|
...nt/agent-debugger/debugger-el/src/main/java/com/datadog/debugger/el/EvaluationException.java
Outdated
Show resolved
Hide resolved
dd-java-agent/agent-debugger/src/main/java/com/datadog/debugger/probe/TriggerProbe.java
Show resolved
Hide resolved
another test case for jacoco
What Does This Do
If the distributed debugger is disabled for a service, do not instrument any trigger probe definitions received from the backend.
Motivation
Additional Notes
Contributor Checklist
type:and (comp:orinst:) labels in addition to any usefull labelsclose,fixor any linking keywords when referencing an issue.Use
solvesinstead, and assign the PR milestone to the issueJira ticket: DEBUG-3547