-
Notifications
You must be signed in to change notification settings - Fork 278
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
Add tracer log file to tracer flare if datadog.slf4j.simpleLogger.logFile is defined #6999
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
BenchmarksStartupParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 48 metrics, 15 unstable metrics. Startup time reports for insecure-bankgantt
title insecure-bank - global startup overhead: candidate=1.35.0-SNAPSHOT~a00b66ff59, baseline=1.35.0-SNAPSHOT~d0bd085303
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.086 s) : 0, 1085597
Total [baseline] (8.56 s) : 0, 8560105
Agent [candidate] (1.077 s) : 0, 1076895
Total [candidate] (8.564 s) : 0, 8564475
section iast
Agent [baseline] (1.205 s) : 0, 1205324
Total [baseline] (9.017 s) : 0, 9017435
Agent [candidate] (1.204 s) : 0, 1203587
Total [candidate] (9.026 s) : 0, 9025938
section iast_HARDCODED_SECRET_DISABLED
Agent [baseline] (1.203 s) : 0, 1202747
Total [baseline] (9.001 s) : 0, 9000788
Agent [candidate] (1.211 s) : 0, 1210909
Total [candidate] (9.002 s) : 0, 9001886
section iast_TELEMETRY_OFF
Agent [baseline] (1.203 s) : 0, 1203327
Total [baseline] (9.039 s) : 0, 9038895
Agent [candidate] (1.198 s) : 0, 1197952
Total [candidate] (9.014 s) : 0, 9013900
gantt
title insecure-bank - break down per module: candidate=1.35.0-SNAPSHOT~a00b66ff59, baseline=1.35.0-SNAPSHOT~d0bd085303
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (680.43 ms) : 0, 680430
BytebuddyAgent [candidate] (673.702 ms) : 0, 673702
GlobalTracer [baseline] (312.598 ms) : 0, 312598
GlobalTracer [candidate] (310.814 ms) : 0, 310814
AppSec [baseline] (49.68 ms) : 0, 49680
AppSec [candidate] (49.775 ms) : 0, 49775
Remote Config [baseline] (658.776 µs) : 0, 659
Remote Config [candidate] (665.276 µs) : 0, 665
Telemetry [baseline] (7.593 ms) : 0, 7593
Telemetry [candidate] (7.575 ms) : 0, 7575
section iast
BytebuddyAgent [baseline] (798.018 ms) : 0, 798018
BytebuddyAgent [candidate] (796.567 ms) : 0, 796567
GlobalTracer [baseline] (291.548 ms) : 0, 291548
GlobalTracer [candidate] (291.135 ms) : 0, 291135
AppSec [baseline] (52.122 ms) : 0, 52122
AppSec [candidate] (49.979 ms) : 0, 49979
IAST [baseline] (22.006 ms) : 0, 22006
IAST [candidate] (24.482 ms) : 0, 24482
Remote Config [baseline] (612.726 µs) : 0, 613
Remote Config [candidate] (580.127 µs) : 0, 580
Telemetry [baseline] (6.641 ms) : 0, 6641
Telemetry [candidate] (6.587 ms) : 0, 6587
section iast_HARDCODED_SECRET_DISABLED
BytebuddyAgent [baseline] (796.098 ms) : 0, 796098
BytebuddyAgent [candidate] (802.439 ms) : 0, 802439
GlobalTracer [baseline] (290.775 ms) : 0, 290775
GlobalTracer [candidate] (292.518 ms) : 0, 292518
AppSec [baseline] (51.387 ms) : 0, 51387
AppSec [candidate] (49.539 ms) : 0, 49539
IAST [baseline] (22.904 ms) : 0, 22904
IAST [candidate] (24.468 ms) : 0, 24468
Remote Config [baseline] (612.538 µs) : 0, 613
Remote Config [candidate] (649.732 µs) : 0, 650
Telemetry [baseline] (6.676 ms) : 0, 6676
Telemetry [candidate] (6.682 ms) : 0, 6682
section iast_TELEMETRY_OFF
BytebuddyAgent [baseline] (795.861 ms) : 0, 795861
BytebuddyAgent [candidate] (791.291 ms) : 0, 791291
GlobalTracer [baseline] (291.765 ms) : 0, 291765
GlobalTracer [candidate] (290.466 ms) : 0, 290466
AppSec [baseline] (51.644 ms) : 0, 51644
AppSec [candidate] (51.878 ms) : 0, 51878
IAST [baseline] (21.78 ms) : 0, 21780
IAST [candidate] (22.871 ms) : 0, 22871
Remote Config [baseline] (611.977 µs) : 0, 612
Remote Config [candidate] (619.492 µs) : 0, 619
Telemetry [baseline] (7.237 ms) : 0, 7237
Telemetry [candidate] (6.559 ms) : 0, 6559
Startup time reports for petclinicgantt
title petclinic - global startup overhead: candidate=1.35.0-SNAPSHOT~a00b66ff59, baseline=1.35.0-SNAPSHOT~d0bd085303
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.076 s) : 0, 1076343
Total [baseline] (10.437 s) : 0, 10436681
Agent [candidate] (1.08 s) : 0, 1080248
Total [candidate] (10.401 s) : 0, 10401111
section appsec
Agent [baseline] (1.207 s) : 0, 1207494
Total [baseline] (10.552 s) : 0, 10552466
Agent [candidate] (1.198 s) : 0, 1198145
Total [candidate] (10.53 s) : 0, 10530155
section iast
Agent [baseline] (1.209 s) : 0, 1208648
Total [baseline] (10.745 s) : 0, 10744856
Agent [candidate] (1.208 s) : 0, 1208403
Total [candidate] (10.791 s) : 0, 10790586
section profiling
Agent [baseline] (1.272 s) : 0, 1272131
Total [baseline] (10.662 s) : 0, 10662340
Agent [candidate] (1.279 s) : 0, 1278650
Total [candidate] (10.619 s) : 0, 10618989
gantt
title petclinic - break down per module: candidate=1.35.0-SNAPSHOT~a00b66ff59, baseline=1.35.0-SNAPSHOT~d0bd085303
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (674.123 ms) : 0, 674123
BytebuddyAgent [candidate] (675.739 ms) : 0, 675739
GlobalTracer [baseline] (309.956 ms) : 0, 309956
GlobalTracer [candidate] (311.777 ms) : 0, 311777
AppSec [baseline] (49.697 ms) : 0, 49697
AppSec [candidate] (49.965 ms) : 0, 49965
Remote Config [baseline] (664.43 µs) : 0, 664
Remote Config [candidate] (665.848 µs) : 0, 666
Telemetry [baseline] (7.64 ms) : 0, 7640
Telemetry [candidate] (7.585 ms) : 0, 7585
section appsec
BytebuddyAgent [baseline] (704.017 ms) : 0, 704017
BytebuddyAgent [candidate] (697.949 ms) : 0, 697949
GlobalTracer [baseline] (296.476 ms) : 0, 296476
GlobalTracer [candidate] (294.565 ms) : 0, 294565
AppSec [baseline] (153.183 ms) : 0, 153183
AppSec [candidate] (152.808 ms) : 0, 152808
Remote Config [baseline] (625.559 µs) : 0, 626
Remote Config [candidate] (623.601 µs) : 0, 624
Telemetry [baseline] (8.852 ms) : 0, 8852
Telemetry [candidate] (8.199 ms) : 0, 8199
IAST [baseline] (19.063 ms) : 0, 19063
IAST [candidate] (19.054 ms) : 0, 19054
section iast
BytebuddyAgent [baseline] (799.731 ms) : 0, 799731
BytebuddyAgent [candidate] (799.811 ms) : 0, 799811
GlobalTracer [baseline] (292.16 ms) : 0, 292160
GlobalTracer [candidate] (291.873 ms) : 0, 291873
AppSec [baseline] (50.473 ms) : 0, 50473
AppSec [candidate] (50.485 ms) : 0, 50485
Remote Config [baseline] (616.671 µs) : 0, 617
Remote Config [candidate] (590.175 µs) : 0, 590
Telemetry [baseline] (6.616 ms) : 0, 6616
Telemetry [candidate] (8.137 ms) : 0, 8137
IAST [baseline] (24.594 ms) : 0, 24594
IAST [candidate] (23.0 ms) : 0, 23000
section profiling
ProfilingAgent [baseline] (95.986 ms) : 0, 95986
ProfilingAgent [candidate] (96.18 ms) : 0, 96180
BytebuddyAgent [baseline] (678.392 ms) : 0, 678392
BytebuddyAgent [candidate] (682.72 ms) : 0, 682720
GlobalTracer [baseline] (382.96 ms) : 0, 382960
GlobalTracer [candidate] (384.09 ms) : 0, 384090
AppSec [baseline] (50.139 ms) : 0, 50139
AppSec [candidate] (50.53 ms) : 0, 50530
Remote Config [baseline] (696.787 µs) : 0, 697
Remote Config [candidate] (708.943 µs) : 0, 709
Telemetry [baseline] (7.484 ms) : 0, 7484
Telemetry [candidate] (7.539 ms) : 0, 7539
Profiling [baseline] (96.01 ms) : 0, 96010
Profiling [candidate] (96.204 ms) : 0, 96204
LoadParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 11 metrics, 17 unstable metrics. Request duration reports for petclinicgantt
title petclinic - request duration [CI 0.99] : candidate=1.35.0-SNAPSHOT~a00b66ff59, baseline=1.35.0-SNAPSHOT~d0bd085303
dateFormat X
axisFormat %s
section baseline
no_agent (1.333 ms) : 1312, 1353
. : milestone, 1333,
appsec (1.716 ms) : 1694, 1739
. : milestone, 1716,
appsec_no_iast (1.669 ms) : 1645, 1693
. : milestone, 1669,
iast (1.465 ms) : 1442, 1487
. : milestone, 1465,
profiling (1.478 ms) : 1453, 1503
. : milestone, 1478,
tracing (1.464 ms) : 1439, 1489
. : milestone, 1464,
section candidate
no_agent (1.338 ms) : 1319, 1358
. : milestone, 1338,
appsec (1.734 ms) : 1711, 1758
. : milestone, 1734,
appsec_no_iast (1.72 ms) : 1696, 1744
. : milestone, 1720,
iast (1.468 ms) : 1446, 1491
. : milestone, 1468,
profiling (1.486 ms) : 1462, 1511
. : milestone, 1486,
tracing (1.468 ms) : 1445, 1492
. : milestone, 1468,
Request duration reports for insecure-bankgantt
title insecure-bank - request duration [CI 0.99] : candidate=1.35.0-SNAPSHOT~a00b66ff59, baseline=1.35.0-SNAPSHOT~d0bd085303
dateFormat X
axisFormat %s
section baseline
no_agent (369.884 µs) : 348, 392
. : milestone, 370,
iast (474.663 µs) : 454, 496
. : milestone, 475,
iast_FULL (547.16 µs) : 526, 568
. : milestone, 547,
iast_GLOBAL (498.886 µs) : 478, 520
. : milestone, 499,
iast_HARDCODED_SECRET_DISABLED (481.191 µs) : 459, 504
. : milestone, 481,
iast_INACTIVE (444.655 µs) : 424, 466
. : milestone, 445,
iast_TELEMETRY_OFF (465.284 µs) : 445, 486
. : milestone, 465,
tracing (440.898 µs) : 420, 462
. : milestone, 441,
section candidate
no_agent (358.754 µs) : 339, 378
. : milestone, 359,
iast (473.808 µs) : 453, 495
. : milestone, 474,
iast_FULL (547.38 µs) : 526, 569
. : milestone, 547,
iast_GLOBAL (505.574 µs) : 485, 527
. : milestone, 506,
iast_HARDCODED_SECRET_DISABLED (475.78 µs) : 455, 497
. : milestone, 476,
iast_INACTIVE (446.72 µs) : 426, 468
. : milestone, 447,
iast_TELEMETRY_OFF (466.223 µs) : 445, 487
. : milestone, 466,
tracing (444.176 µs) : 423, 466
. : milestone, 444,
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 biojavagantt
title biojava - execution time [CI 0.99] : candidate=1.35.0-SNAPSHOT~a00b66ff59, baseline=1.35.0-SNAPSHOT~d0bd085303
dateFormat X
axisFormat %s
section baseline
no_agent (15.036 s) : 15036000, 15036000
. : milestone, 15036000,
appsec (15.022 s) : 15022000, 15022000
. : milestone, 15022000,
iast (18.874 s) : 18874000, 18874000
. : milestone, 18874000,
iast_GLOBAL (17.883 s) : 17883000, 17883000
. : milestone, 17883000,
profiling (14.916 s) : 14916000, 14916000
. : milestone, 14916000,
tracing (14.804 s) : 14804000, 14804000
. : milestone, 14804000,
section candidate
no_agent (14.918 s) : 14918000, 14918000
. : milestone, 14918000,
appsec (14.984 s) : 14984000, 14984000
. : milestone, 14984000,
iast (19.064 s) : 19064000, 19064000
. : milestone, 19064000,
iast_GLOBAL (17.814 s) : 17814000, 17814000
. : milestone, 17814000,
profiling (15.012 s) : 15012000, 15012000
. : milestone, 15012000,
tracing (14.957 s) : 14957000, 14957000
. : milestone, 14957000,
Execution time for tomcatgantt
title tomcat - execution time [CI 0.99] : candidate=1.35.0-SNAPSHOT~a00b66ff59, baseline=1.35.0-SNAPSHOT~d0bd085303
dateFormat X
axisFormat %s
section baseline
no_agent (1.467 ms) : 1456, 1478
. : milestone, 1467,
appsec (2.205 ms) : 2172, 2239
. : milestone, 2205,
iast (1.964 ms) : 1923, 2005
. : milestone, 1964,
iast_GLOBAL (2.0 ms) : 1959, 2041
. : milestone, 2000,
profiling (1.855 ms) : 1821, 1889
. : milestone, 1855,
tracing (1.835 ms) : 1802, 1867
. : milestone, 1835,
section candidate
no_agent (1.46 ms) : 1449, 1472
. : milestone, 1460,
appsec (2.21 ms) : 2176, 2244
. : milestone, 2210,
iast (1.964 ms) : 1923, 2005
. : milestone, 1964,
iast_GLOBAL (1.984 ms) : 1943, 2024
. : milestone, 1984,
profiling (1.84 ms) : 1808, 1872
. : milestone, 1840,
tracing (1.837 ms) : 1805, 1869
. : milestone, 1837,
|
8e3ea7e
to
47112ba
Compare
dougqh
approved these changes
May 15, 2024
mcculls
approved these changes
May 17, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
What Does This Do
This PR is a follow up of
This PR adds tracer log file to tracer flare, requested by the UI or by Triage mode:
if a tracer log file has been defined with datadog.slf4j.simpleLogger.logFile and if the size of the tracer log file is > 15MB:
It's likely that the last line of "tracer_begin.log" and the first line of "tracer_end.log" will be incomplete. To only handling bytes in the java code, it will stay as it is but any tool to automatically analyze tracer debug log should discard the last line of "tracer_begin.log" and the first line of "tracer_end.log" are they are likely to be incomplete.
Motivation
Getting tracer logs in tracer flare to get as much information as possible to better troubleshoot issues
Additional Notes
Jira ticket: [APMAPI-59](https://datadoghq.atlassian.net/browse/APMAPI-59)