-
Notifications
You must be signed in to change notification settings - Fork 320
Send only snapshots to Debugger Track #9501
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 0 performance regressions! Performance is the same for 9 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 (329.318 µs) : 285, 374
. : milestone, 329,
basic (280.906 µs) : 274, 288
. : milestone, 281,
loop (8.96 ms) : 8955, 8965
. : milestone, 8960,
section candidate
noprobe (334.796 µs) : 277, 393
. : milestone, 335,
basic (279.657 µs) : 273, 286
. : milestone, 280,
loop (8.964 ms) : 8959, 8969
. : milestone, 8964,
|
|
🎯 Code Coverage 🔗 Commit SHA: 0d9742a | Docs | Was this helpful? Give us feedback! |
BenchmarksStartupParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 48 metrics, 11 unstable metrics. Startup time reports for petclinicgantt
title petclinic - global startup overhead: candidate=1.54.0-SNAPSHOT~0d9742aa73, baseline=1.54.0-SNAPSHOT~036fe910ec
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.058 s) : 0, 1058406
Total [baseline] (10.806 s) : 0, 10806497
Agent [candidate] (1.046 s) : 0, 1045899
Total [candidate] (10.662 s) : 0, 10662470
section appsec
Agent [baseline] (1.232 s) : 0, 1232011
Total [baseline] (10.907 s) : 0, 10907026
Agent [candidate] (1.228 s) : 0, 1227718
Total [candidate] (10.863 s) : 0, 10862754
section iast
Agent [baseline] (1.189 s) : 0, 1188675
Total [baseline] (11.005 s) : 0, 11005266
Agent [candidate] (1.192 s) : 0, 1191513
Total [candidate] (10.951 s) : 0, 10951134
section profiling
Agent [baseline] (1.201 s) : 0, 1201183
Total [baseline] (10.977 s) : 0, 10977047
Agent [candidate] (1.202 s) : 0, 1202229
Total [candidate] (10.931 s) : 0, 10930833
gantt
title petclinic - break down per module: candidate=1.54.0-SNAPSHOT~0d9742aa73, baseline=1.54.0-SNAPSHOT~036fe910ec
dateFormat X
axisFormat %s
section tracing
crashtracking [baseline] (1.452 ms) : 0, 1452
crashtracking [candidate] (1.445 ms) : 0, 1445
BytebuddyAgent [baseline] (740.138 ms) : 0, 740138
BytebuddyAgent [candidate] (732.832 ms) : 0, 732832
GlobalTracer [baseline] (245.228 ms) : 0, 245228
GlobalTracer [candidate] (242.805 ms) : 0, 242805
AppSec [baseline] (30.872 ms) : 0, 30872
AppSec [candidate] (30.497 ms) : 0, 30497
Debugger [baseline] (6.515 ms) : 0, 6515
Debugger [candidate] (6.422 ms) : 0, 6422
Remote Config [baseline] (682.327 µs) : 0, 682
Remote Config [candidate] (676.375 µs) : 0, 676
Telemetry [baseline] (12.394 ms) : 0, 12394
Telemetry [candidate] (10.152 ms) : 0, 10152
section appsec
crashtracking [baseline] (1.469 ms) : 0, 1469
crashtracking [candidate] (1.453 ms) : 0, 1453
BytebuddyAgent [baseline] (760.615 ms) : 0, 760615
BytebuddyAgent [candidate] (757.701 ms) : 0, 757701
GlobalTracer [baseline] (237.098 ms) : 0, 237098
GlobalTracer [candidate] (236.394 ms) : 0, 236394
IAST [baseline] (23.836 ms) : 0, 23836
IAST [candidate] (23.676 ms) : 0, 23676
AppSec [baseline] (170.015 ms) : 0, 170015
AppSec [candidate] (170.0 ms) : 0, 170000
Debugger [baseline] (7.669 ms) : 0, 7669
Debugger [candidate] (7.579 ms) : 0, 7579
Remote Config [baseline] (641.602 µs) : 0, 642
Remote Config [candidate] (672.873 µs) : 0, 673
Telemetry [baseline] (9.43 ms) : 0, 9430
Telemetry [candidate] (9.193 ms) : 0, 9193
section iast
crashtracking [baseline] (1.456 ms) : 0, 1456
crashtracking [candidate] (1.471 ms) : 0, 1471
BytebuddyAgent [baseline] (857.967 ms) : 0, 857967
BytebuddyAgent [candidate] (860.053 ms) : 0, 860053
GlobalTracer [baseline] (234.298 ms) : 0, 234298
GlobalTracer [candidate] (236.037 ms) : 0, 236037
IAST [baseline] (29.534 ms) : 0, 29534
IAST [candidate] (31.431 ms) : 0, 31431
AppSec [baseline] (28.345 ms) : 0, 28345
AppSec [candidate] (26.361 ms) : 0, 26361
Debugger [baseline] (7.009 ms) : 0, 7009
Debugger [candidate] (6.142 ms) : 0, 6142
Remote Config [baseline] (620.143 µs) : 0, 620
Remote Config [candidate] (608.281 µs) : 0, 608
Telemetry [baseline] (8.301 ms) : 0, 8301
Telemetry [candidate] (8.24 ms) : 0, 8240
section profiling
crashtracking [baseline] (1.437 ms) : 0, 1437
crashtracking [candidate] (1.421 ms) : 0, 1421
BytebuddyAgent [baseline] (762.572 ms) : 0, 762572
BytebuddyAgent [candidate] (763.75 ms) : 0, 763750
GlobalTracer [baseline] (223.397 ms) : 0, 223397
GlobalTracer [candidate] (223.937 ms) : 0, 223937
AppSec [baseline] (31.006 ms) : 0, 31006
AppSec [candidate] (30.944 ms) : 0, 30944
Debugger [baseline] (6.634 ms) : 0, 6634
Debugger [candidate] (6.754 ms) : 0, 6754
Remote Config [baseline] (692.11 µs) : 0, 692
Remote Config [candidate] (724.848 µs) : 0, 725
Telemetry [baseline] (16.64 ms) : 0, 16640
Telemetry [candidate] (16.46 ms) : 0, 16460
ProfilingAgent [baseline] (108.089 ms) : 0, 108089
ProfilingAgent [candidate] (107.711 ms) : 0, 107711
Profiling [baseline] (108.822 ms) : 0, 108822
Profiling [candidate] (108.42 ms) : 0, 108420
Startup time reports for insecure-bankgantt
title insecure-bank - global startup overhead: candidate=1.54.0-SNAPSHOT~0d9742aa73, baseline=1.54.0-SNAPSHOT~036fe910ec
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.064 s) : 0, 1064408
Total [baseline] (8.692 s) : 0, 8692125
Agent [candidate] (1.063 s) : 0, 1062573
Total [candidate] (8.665 s) : 0, 8665449
section iast
Agent [baseline] (1.181 s) : 0, 1181417
Total [baseline] (9.312 s) : 0, 9312370
Agent [candidate] (1.183 s) : 0, 1183389
Total [candidate] (9.382 s) : 0, 9382316
gantt
title insecure-bank - break down per module: candidate=1.54.0-SNAPSHOT~0d9742aa73, baseline=1.54.0-SNAPSHOT~036fe910ec
dateFormat X
axisFormat %s
section tracing
crashtracking [baseline] (1.479 ms) : 0, 1479
crashtracking [candidate] (1.474 ms) : 0, 1474
BytebuddyAgent [baseline] (744.142 ms) : 0, 744142
BytebuddyAgent [candidate] (742.435 ms) : 0, 742435
GlobalTracer [baseline] (245.194 ms) : 0, 245194
GlobalTracer [candidate] (245.293 ms) : 0, 245293
AppSec [baseline] (31.11 ms) : 0, 31110
AppSec [candidate] (30.899 ms) : 0, 30899
Debugger [baseline] (6.518 ms) : 0, 6518
Debugger [candidate] (6.503 ms) : 0, 6503
Remote Config [baseline] (692.77 µs) : 0, 693
Remote Config [candidate] (685.036 µs) : 0, 685
Telemetry [baseline] (13.947 ms) : 0, 13947
Telemetry [candidate] (14.021 ms) : 0, 14021
section iast
crashtracking [baseline] (1.46 ms) : 0, 1460
crashtracking [candidate] (1.459 ms) : 0, 1459
BytebuddyAgent [baseline] (852.508 ms) : 0, 852508
BytebuddyAgent [candidate] (854.116 ms) : 0, 854116
GlobalTracer [baseline] (234.932 ms) : 0, 234932
GlobalTracer [candidate] (233.567 ms) : 0, 233567
AppSec [baseline] (27.909 ms) : 0, 27909
AppSec [candidate] (29.463 ms) : 0, 29463
Debugger [baseline] (6.075 ms) : 0, 6075
Debugger [candidate] (6.136 ms) : 0, 6136
Remote Config [baseline] (600.744 µs) : 0, 601
Remote Config [candidate] (598.895 µs) : 0, 599
Telemetry [baseline] (8.286 ms) : 0, 8286
Telemetry [candidate] (8.354 ms) : 0, 8354
IAST [baseline] (28.682 ms) : 0, 28682
IAST [candidate] (28.661 ms) : 0, 28661
LoadParameters
See matching parameters
SummaryFound 1 performance improvements and 1 performance regressions! Performance is the same for 10 metrics, 12 unstable metrics.
Request duration reports for insecure-bankgantt
title insecure-bank - request duration [CI 0.99] : candidate=1.54.0-SNAPSHOT~0d9742aa73, baseline=1.54.0-SNAPSHOT~036fe910ec
dateFormat X
axisFormat %s
section baseline
no_agent (4.218 ms) : 4170, 4266
. : milestone, 4218,
iast (9.511 ms) : 9355, 9668
. : milestone, 9511,
iast_FULL (14.119 ms) : 13841, 14397
. : milestone, 14119,
iast_GLOBAL (10.753 ms) : 10552, 10955
. : milestone, 10753,
profiling (8.679 ms) : 8542, 8817
. : milestone, 8679,
tracing (7.802 ms) : 7682, 7921
. : milestone, 7802,
section candidate
no_agent (4.265 ms) : 4214, 4316
. : milestone, 4265,
iast (9.438 ms) : 9283, 9594
. : milestone, 9438,
iast_FULL (13.707 ms) : 13435, 13978
. : milestone, 13707,
iast_GLOBAL (10.538 ms) : 10349, 10727
. : milestone, 10538,
profiling (9.133 ms) : 8981, 9285
. : milestone, 9133,
tracing (7.777 ms) : 7664, 7890
. : milestone, 7777,
Request duration reports for petclinicgantt
title petclinic - request duration [CI 0.99] : candidate=1.54.0-SNAPSHOT~0d9742aa73, baseline=1.54.0-SNAPSHOT~036fe910ec
dateFormat X
axisFormat %s
section baseline
no_agent (36.919 ms) : 36614, 37224
. : milestone, 36919,
appsec (49.509 ms) : 49055, 49963
. : milestone, 49509,
code_origins (45.143 ms) : 44756, 45530
. : milestone, 45143,
iast (45.494 ms) : 45089, 45899
. : milestone, 45494,
profiling (49.305 ms) : 48801, 49808
. : milestone, 49305,
tracing (42.748 ms) : 42402, 43095
. : milestone, 42748,
section candidate
no_agent (36.758 ms) : 36456, 37060
. : milestone, 36758,
appsec (48.578 ms) : 48162, 48993
. : milestone, 48578,
code_origins (46.409 ms) : 46001, 46816
. : milestone, 46409,
iast (45.109 ms) : 44716, 45501
. : milestone, 45109,
profiling (47.665 ms) : 47244, 48087
. : milestone, 47665,
tracing (43.915 ms) : 43551, 44280
. : milestone, 43915,
DacapoParameters
See matching parameters
SummaryFound 1 performance improvements and 0 performance regressions! Performance is the same for 11 metrics, 0 unstable metrics.
Execution time for biojavagantt
title biojava - execution time [CI 0.99] : candidate=1.54.0-SNAPSHOT~0d9742aa73, baseline=1.54.0-SNAPSHOT~036fe910ec
dateFormat X
axisFormat %s
section baseline
no_agent (15.619 s) : 15619000, 15619000
. : milestone, 15619000,
appsec (15.021 s) : 15021000, 15021000
. : milestone, 15021000,
iast (18.205 s) : 18205000, 18205000
. : milestone, 18205000,
iast_GLOBAL (18.03 s) : 18030000, 18030000
. : milestone, 18030000,
profiling (15.763 s) : 15763000, 15763000
. : milestone, 15763000,
tracing (14.923 s) : 14923000, 14923000
. : milestone, 14923000,
section candidate
no_agent (15.02 s) : 15020000, 15020000
. : milestone, 15020000,
appsec (15.034 s) : 15034000, 15034000
. : milestone, 15034000,
iast (18.36 s) : 18360000, 18360000
. : milestone, 18360000,
iast_GLOBAL (17.941 s) : 17941000, 17941000
. : milestone, 17941000,
profiling (15.719 s) : 15719000, 15719000
. : milestone, 15719000,
tracing (14.956 s) : 14956000, 14956000
. : milestone, 14956000,
Execution time for tomcatgantt
title tomcat - execution time [CI 0.99] : candidate=1.54.0-SNAPSHOT~0d9742aa73, baseline=1.54.0-SNAPSHOT~036fe910ec
dateFormat X
axisFormat %s
section baseline
no_agent (1.47 ms) : 1458, 1481
. : milestone, 1470,
appsec (3.712 ms) : 3495, 3930
. : milestone, 3712,
iast (2.198 ms) : 2135, 2261
. : milestone, 2198,
iast_GLOBAL (2.238 ms) : 2175, 2302
. : milestone, 2238,
profiling (2.055 ms) : 2003, 2107
. : milestone, 2055,
tracing (2.004 ms) : 1956, 2053
. : milestone, 2004,
section candidate
no_agent (1.47 ms) : 1459, 1482
. : milestone, 1470,
appsec (2.495 ms) : 2442, 2547
. : milestone, 2495,
iast (2.181 ms) : 2118, 2243
. : milestone, 2181,
iast_GLOBAL (2.237 ms) : 2174, 2300
. : milestone, 2237,
profiling (2.041 ms) : 1990, 2091
. : milestone, 2041,
tracing (2.003 ms) : 1955, 2052
. : milestone, 2003,
|
f289477 to
cc0281a
Compare
| SnapshotSink.RETRY_POLICY), | ||
| new BatchUploader( | ||
| "Logs", | ||
| config, | ||
| config.getFinalDebuggerSnapshotUrl(), | ||
| SnapshotSink.RETRY_POLICY)), |
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.
I can see that we are using RETRY_POLICY always? Can it be some sort of default?
Also I have a feeling that RETRY_POLICY should be in BatchUploader class - WDYT?
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.
for SymDB the policy is different see
dd-trace-java/dd-java-agent/agent-debugger/src/main/java/com/datadog/debugger/sink/SymbolSink.java
Line 32 in cb7783b
| public static final BatchUploader.RetryPolicy RETRY_POLICY = new BatchUploader.RetryPolicy(10); |
cc0281a to
ab0fec5
Compare
We introduce a separate BatchUploader instance to send capureSnapshot=true snapshots (including variables) to dedicated datadog endpoint for DEBUGGER track. SnapshotSink now expect 2 BatchUploaders (lowRate for snapshots and highRate for templated logs only) BatchUploader now take a name to identify in debug logs which instance is used. Smoke tests are updated to use processRequest everywhere and get rid of retrieveSnapshots that is not correct.
ab0fec5 to
0d9742a
Compare
What Does This Do
We introduce a separate BatchUploader instance to send capureSnapshot=true snapshots (including variables) to dedicated datadog endpoint for DEBUGGER track.
SnapshotSink now expect 2 BatchUploaders (lowRate for snapshots and highRate for templated logs only)
BatchUploader now take a name to identify in debug logs which instance is used.
Smoke tests are updated to use processRequest everywhere and get rid of retrieveSnapshots that is not correct.
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-4444