-
Notifications
You must be signed in to change notification settings - Fork 292
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
Consult the environment variable when setting the max users frames in code origin probes #7881
Conversation
… code origin probes
… code origin probes
dd-java-agent/agent-debugger/src/main/java/com/datadog/debugger/probe/CodeOriginProbe.java
Outdated
Show resolved
Hide resolved
Debugger benchmarksParameters
See matching parameters
SummaryFound 1 performance improvements and 0 performance regressions! Performance is the same for 9 metrics, 5 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 (340.222 µs) : 312, 369
. : milestone, 340,
basic (347.999 µs) : 337, 359
. : milestone, 348,
loop (11.517 ms) : 11480, 11555
. : milestone, 11517,
section candidate
noprobe (339.481 µs) : 309, 370
. : milestone, 339,
basic (333.933 µs) : 324, 344
. : milestone, 334,
loop (11.462 ms) : 11413, 11510
. : milestone, 11462,
|
BenchmarksStartupParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 53 metrics, 10 unstable metrics. Startup time reports for petclinicgantt
title petclinic - global startup overhead: candidate=1.43.0-SNAPSHOT~ce240a966c, baseline=1.43.0-SNAPSHOT~f49ac98c70
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.087 s) : 0, 1087239
Total [baseline] (10.401 s) : 0, 10401300
Agent [candidate] (1.083 s) : 0, 1083111
Total [candidate] (10.406 s) : 0, 10405820
section appsec
Agent [baseline] (1.215 s) : 0, 1214824
Total [baseline] (10.74 s) : 0, 10739817
Agent [candidate] (1.221 s) : 0, 1220991
Total [candidate] (10.701 s) : 0, 10701095
section iast
Agent [baseline] (1.208 s) : 0, 1208223
Total [baseline] (10.943 s) : 0, 10942899
Agent [candidate] (1.226 s) : 0, 1226209
Total [candidate] (11.023 s) : 0, 11023417
section profiling
Agent [baseline] (1.28 s) : 0, 1279532
Total [baseline] (10.779 s) : 0, 10779318
Agent [candidate] (1.282 s) : 0, 1281958
Total [candidate] (10.722 s) : 0, 10721594
gantt
title petclinic - break down per module: candidate=1.43.0-SNAPSHOT~ce240a966c, baseline=1.43.0-SNAPSHOT~f49ac98c70
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (692.644 ms) : 0, 692644
BytebuddyAgent [candidate] (687.377 ms) : 0, 687377
GlobalTracer [baseline] (316.188 ms) : 0, 316188
GlobalTracer [candidate] (315.822 ms) : 0, 315822
AppSec [baseline] (54.175 ms) : 0, 54175
AppSec [candidate] (54.319 ms) : 0, 54319
Remote Config [baseline] (662.832 µs) : 0, 663
Remote Config [candidate] (664.276 µs) : 0, 664
Telemetry [baseline] (9.757 ms) : 0, 9757
Telemetry [candidate] (11.218 ms) : 0, 11218
section appsec
BytebuddyAgent [baseline] (704.193 ms) : 0, 704193
BytebuddyAgent [candidate] (708.987 ms) : 0, 708987
GlobalTracer [baseline] (312.935 ms) : 0, 312935
GlobalTracer [candidate] (314.122 ms) : 0, 314122
AppSec [baseline] (165.206 ms) : 0, 165206
AppSec [candidate] (165.189 ms) : 0, 165189
IAST [baseline] (19.438 ms) : 0, 19438
IAST [candidate] (20.885 ms) : 0, 20885
Remote Config [baseline] (639.789 µs) : 0, 640
Remote Config [candidate] (641.863 µs) : 0, 642
Telemetry [baseline] (8.433 ms) : 0, 8433
Telemetry [candidate] (7.819 ms) : 0, 7819
section iast
BytebuddyAgent [baseline] (803.281 ms) : 0, 803281
BytebuddyAgent [candidate] (816.346 ms) : 0, 816346
GlobalTracer [baseline] (304.911 ms) : 0, 304911
GlobalTracer [candidate] (308.824 ms) : 0, 308824
AppSec [baseline] (56.861 ms) : 0, 56861
AppSec [candidate] (57.912 ms) : 0, 57912
IAST [baseline] (21.337 ms) : 0, 21337
IAST [candidate] (20.958 ms) : 0, 20958
Remote Config [baseline] (617.223 µs) : 0, 617
Remote Config [candidate] (628.585 µs) : 0, 629
Telemetry [baseline] (7.454 ms) : 0, 7454
Telemetry [candidate] (7.574 ms) : 0, 7574
section profiling
BytebuddyAgent [baseline] (680.447 ms) : 0, 680447
BytebuddyAgent [candidate] (683.009 ms) : 0, 683009
GlobalTracer [baseline] (400.219 ms) : 0, 400219
GlobalTracer [candidate] (399.241 ms) : 0, 399241
AppSec [baseline] (54.632 ms) : 0, 54632
AppSec [candidate] (54.674 ms) : 0, 54674
Remote Config [baseline] (682.962 µs) : 0, 683
Remote Config [candidate] (681.968 µs) : 0, 682
Telemetry [baseline] (12.084 ms) : 0, 12084
Telemetry [candidate] (10.088 ms) : 0, 10088
ProfilingAgent [baseline] (92.472 ms) : 0, 92472
ProfilingAgent [candidate] (95.174 ms) : 0, 95174
Profiling [baseline] (92.496 ms) : 0, 92496
Profiling [candidate] (95.199 ms) : 0, 95199
Startup time reports for insecure-bankgantt
title insecure-bank - global startup overhead: candidate=1.43.0-SNAPSHOT~ce240a966c, baseline=1.43.0-SNAPSHOT~f49ac98c70
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.085 s) : 0, 1085337
Total [baseline] (8.598 s) : 0, 8598009
Agent [candidate] (1.09 s) : 0, 1089901
Total [candidate] (8.63 s) : 0, 8629680
section iast
Agent [baseline] (1.21 s) : 0, 1210293
Total [baseline] (9.163 s) : 0, 9163411
Agent [candidate] (1.207 s) : 0, 1207222
Total [candidate] (9.167 s) : 0, 9166715
section iast_HARDCODED_SECRET_DISABLED
Agent [baseline] (1.209 s) : 0, 1208983
Total [baseline] (9.14 s) : 0, 9139789
Agent [candidate] (1.208 s) : 0, 1208478
Total [candidate] (9.127 s) : 0, 9127409
section iast_TELEMETRY_OFF
Agent [baseline] (1.211 s) : 0, 1210995
Total [baseline] (9.169 s) : 0, 9168648
Agent [candidate] (1.208 s) : 0, 1208293
Total [candidate] (9.164 s) : 0, 9164254
gantt
title insecure-bank - break down per module: candidate=1.43.0-SNAPSHOT~ce240a966c, baseline=1.43.0-SNAPSHOT~f49ac98c70
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (689.358 ms) : 0, 689358
BytebuddyAgent [candidate] (693.767 ms) : 0, 693767
GlobalTracer [baseline] (316.77 ms) : 0, 316770
GlobalTracer [candidate] (317.907 ms) : 0, 317907
AppSec [baseline] (54.144 ms) : 0, 54144
AppSec [candidate] (54.496 ms) : 0, 54496
Remote Config [baseline] (671.679 µs) : 0, 672
Remote Config [candidate] (662.071 µs) : 0, 662
Telemetry [baseline] (10.602 ms) : 0, 10602
Telemetry [candidate] (9.162 ms) : 0, 9162
section iast
BytebuddyAgent [baseline] (803.159 ms) : 0, 803159
BytebuddyAgent [candidate] (802.625 ms) : 0, 802625
GlobalTracer [baseline] (306.329 ms) : 0, 306329
GlobalTracer [candidate] (304.762 ms) : 0, 304762
AppSec [baseline] (56.265 ms) : 0, 56265
AppSec [candidate] (57.222 ms) : 0, 57222
IAST [baseline] (22.596 ms) : 0, 22596
IAST [candidate] (20.713 ms) : 0, 20713
Remote Config [baseline] (637.676 µs) : 0, 638
Remote Config [candidate] (625.926 µs) : 0, 626
Telemetry [baseline] (7.548 ms) : 0, 7548
Telemetry [candidate] (7.523 ms) : 0, 7523
section iast_HARDCODED_SECRET_DISABLED
BytebuddyAgent [baseline] (803.869 ms) : 0, 803869
BytebuddyAgent [candidate] (802.996 ms) : 0, 802996
GlobalTracer [baseline] (305.114 ms) : 0, 305114
GlobalTracer [candidate] (304.865 ms) : 0, 304865
AppSec [baseline] (57.314 ms) : 0, 57314
AppSec [candidate] (56.534 ms) : 0, 56534
IAST [baseline] (20.771 ms) : 0, 20771
IAST [candidate] (22.137 ms) : 0, 22137
Remote Config [baseline] (625.174 µs) : 0, 625
Remote Config [candidate] (614.626 µs) : 0, 615
Telemetry [baseline] (7.509 ms) : 0, 7509
Telemetry [candidate] (7.558 ms) : 0, 7558
section iast_TELEMETRY_OFF
BytebuddyAgent [baseline] (804.224 ms) : 0, 804224
BytebuddyAgent [candidate] (801.914 ms) : 0, 801914
GlobalTracer [baseline] (306.41 ms) : 0, 306410
GlobalTracer [candidate] (305.927 ms) : 0, 305927
AppSec [baseline] (57.925 ms) : 0, 57925
AppSec [candidate] (58.197 ms) : 0, 58197
IAST [baseline] (20.507 ms) : 0, 20507
IAST [candidate] (20.367 ms) : 0, 20367
Remote Config [baseline] (631.256 µs) : 0, 631
Remote Config [candidate] (616.595 µs) : 0, 617
Telemetry [baseline] (7.489 ms) : 0, 7489
Telemetry [candidate] (7.474 ms) : 0, 7474
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.43.0-SNAPSHOT~ce240a966c, baseline=1.43.0-SNAPSHOT~f49ac98c70
dateFormat X
axisFormat %s
section baseline
no_agent (1.347 ms) : 1327, 1368
. : milestone, 1347,
appsec (1.737 ms) : 1711, 1762
. : milestone, 1737,
appsec_no_iast (1.737 ms) : 1712, 1762
. : milestone, 1737,
iast (1.487 ms) : 1464, 1509
. : milestone, 1487,
profiling (1.511 ms) : 1488, 1534
. : milestone, 1511,
tracing (1.468 ms) : 1442, 1493
. : milestone, 1468,
section candidate
no_agent (1.347 ms) : 1327, 1367
. : milestone, 1347,
appsec (1.733 ms) : 1708, 1757
. : milestone, 1733,
appsec_no_iast (1.745 ms) : 1721, 1769
. : milestone, 1745,
iast (1.485 ms) : 1462, 1508
. : milestone, 1485,
profiling (1.504 ms) : 1481, 1527
. : milestone, 1504,
tracing (1.489 ms) : 1465, 1513
. : milestone, 1489,
Request duration reports for insecure-bankgantt
title insecure-bank - request duration [CI 0.99] : candidate=1.43.0-SNAPSHOT~ce240a966c, baseline=1.43.0-SNAPSHOT~f49ac98c70
dateFormat X
axisFormat %s
section baseline
no_agent (376.151 µs) : 356, 396
. : milestone, 376,
iast (499.717 µs) : 478, 521
. : milestone, 500,
iast_FULL (644.259 µs) : 623, 666
. : milestone, 644,
iast_GLOBAL (513.153 µs) : 492, 534
. : milestone, 513,
iast_HARDCODED_SECRET_DISABLED (493.323 µs) : 472, 515
. : milestone, 493,
iast_INACTIVE (456.35 µs) : 435, 478
. : milestone, 456,
iast_TELEMETRY_OFF (486.15 µs) : 464, 508
. : milestone, 486,
tracing (455.634 µs) : 435, 477
. : milestone, 456,
section candidate
no_agent (368.845 µs) : 349, 389
. : milestone, 369,
iast (486.128 µs) : 465, 507
. : milestone, 486,
iast_FULL (646.976 µs) : 625, 669
. : milestone, 647,
iast_GLOBAL (518.544 µs) : 497, 540
. : milestone, 519,
iast_HARDCODED_SECRET_DISABLED (492.098 µs) : 471, 513
. : milestone, 492,
iast_INACTIVE (455.34 µs) : 434, 477
. : milestone, 455,
iast_TELEMETRY_OFF (480.136 µs) : 459, 502
. : milestone, 480,
tracing (455.261 µs) : 434, 476
. : milestone, 455,
DacapoParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 11 metrics, 1 unstable metrics. Execution time for tomcatgantt
title tomcat - execution time [CI 0.99] : candidate=1.43.0-SNAPSHOT~ce240a966c, baseline=1.43.0-SNAPSHOT~f49ac98c70
dateFormat X
axisFormat %s
section baseline
no_agent (1.468 ms) : 1457, 1480
. : milestone, 1468,
appsec (2.348 ms) : 2306, 2390
. : milestone, 2348,
iast (2.083 ms) : 2031, 2136
. : milestone, 2083,
iast_GLOBAL (2.133 ms) : 2081, 2186
. : milestone, 2133,
profiling (2.394 ms) : 2150, 2638
. : milestone, 2394,
tracing (1.922 ms) : 1882, 1962
. : milestone, 1922,
section candidate
no_agent (1.47 ms) : 1459, 1482
. : milestone, 1470,
appsec (2.347 ms) : 2306, 2389
. : milestone, 2347,
iast (2.085 ms) : 2033, 2138
. : milestone, 2085,
iast_GLOBAL (2.137 ms) : 2083, 2190
. : milestone, 2137,
profiling (1.949 ms) : 1907, 1990
. : milestone, 1949,
tracing (1.944 ms) : 1903, 1985
. : milestone, 1944,
Execution time for biojavagantt
title biojava - execution time [CI 0.99] : candidate=1.43.0-SNAPSHOT~ce240a966c, baseline=1.43.0-SNAPSHOT~f49ac98c70
dateFormat X
axisFormat %s
section baseline
no_agent (15.299 s) : 15299000, 15299000
. : milestone, 15299000,
appsec (15.184 s) : 15184000, 15184000
. : milestone, 15184000,
iast (18.692 s) : 18692000, 18692000
. : milestone, 18692000,
iast_GLOBAL (18.169 s) : 18169000, 18169000
. : milestone, 18169000,
profiling (15.15 s) : 15150000, 15150000
. : milestone, 15150000,
tracing (15.189 s) : 15189000, 15189000
. : milestone, 15189000,
section candidate
no_agent (15.477 s) : 15477000, 15477000
. : milestone, 15477000,
appsec (15.314 s) : 15314000, 15314000
. : milestone, 15314000,
iast (18.97 s) : 18970000, 18970000
. : milestone, 18970000,
iast_GLOBAL (18.17 s) : 18170000, 18170000
. : milestone, 18170000,
profiling (15.256 s) : 15256000, 15256000
. : milestone, 15256000,
tracing (15.418 s) : 15418000, 15418000
. : milestone, 15418000,
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good from Core/Platform POV
What Does This Do
Update the code origin code to check for an environment variable when configuring the maximum number of user frames to include in a code origin probe's output.
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-2727