-
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
Exclude method handles exceptions from telemetry logs #7208
Merged
nikita-tkachenko-datadog
merged 1 commit into
master
from
nikita-tkachenko/ci-vis-telemetry-ignores
Jun 17, 2024
Merged
Exclude method handles exceptions from telemetry logs #7208
nikita-tkachenko-datadog
merged 1 commit into
master
from
nikita-tkachenko/ci-vis-telemetry-ignores
Jun 17, 2024
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
romainkomorndatadog
approved these changes
Jun 17, 2024
BenchmarksStartupParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 52 metrics, 11 unstable metrics. Startup time reports for petclinicgantt
title petclinic - global startup overhead: candidate=1.36.0-SNAPSHOT~44c8c2a5fa, baseline=1.36.0-SNAPSHOT~a19c6d5486
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.064 s) : 0, 1063988
Total [baseline] (10.428 s) : 0, 10427851
Agent [candidate] (1.064 s) : 0, 1064190
Total [candidate] (10.336 s) : 0, 10336188
section appsec
Agent [baseline] (1.186 s) : 0, 1185574
Total [baseline] (10.472 s) : 0, 10471705
Agent [candidate] (1.186 s) : 0, 1185987
Total [candidate] (10.534 s) : 0, 10534080
section iast
Agent [baseline] (1.168 s) : 0, 1167874
Total [baseline] (10.694 s) : 0, 10693584
Agent [candidate] (1.179 s) : 0, 1178884
Total [candidate] (10.815 s) : 0, 10814588
section profiling
Agent [baseline] (1.266 s) : 0, 1265575
Total [baseline] (10.689 s) : 0, 10689110
Agent [candidate] (1.273 s) : 0, 1272634
Total [candidate] (10.702 s) : 0, 10701722
gantt
title petclinic - break down per module: candidate=1.36.0-SNAPSHOT~44c8c2a5fa, baseline=1.36.0-SNAPSHOT~a19c6d5486
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (666.415 ms) : 0, 666415
BytebuddyAgent [candidate] (666.678 ms) : 0, 666678
GlobalTracer [baseline] (304.329 ms) : 0, 304329
GlobalTracer [candidate] (304.509 ms) : 0, 304509
AppSec [baseline] (50.395 ms) : 0, 50395
AppSec [candidate] (50.17 ms) : 0, 50170
Remote Config [baseline] (688.911 µs) : 0, 689
Remote Config [candidate] (685.402 µs) : 0, 685
Telemetry [baseline] (7.692 ms) : 0, 7692
Telemetry [candidate] (7.607 ms) : 0, 7607
section appsec
BytebuddyAgent [baseline] (677.445 ms) : 0, 677445
BytebuddyAgent [candidate] (678.584 ms) : 0, 678584
GlobalTracer [baseline] (297.836 ms) : 0, 297836
GlobalTracer [candidate] (298.51 ms) : 0, 298510
AppSec [baseline] (154.558 ms) : 0, 154558
AppSec [candidate] (154.459 ms) : 0, 154459
IAST [baseline] (23.019 ms) : 0, 23019
IAST [candidate] (20.942 ms) : 0, 20942
Remote Config [baseline] (640.953 µs) : 0, 641
Remote Config [candidate] (644.599 µs) : 0, 645
Telemetry [baseline] (9.058 ms) : 0, 9058
Telemetry [candidate] (8.763 ms) : 0, 8763
section iast
BytebuddyAgent [baseline] (779.181 ms) : 0, 779181
BytebuddyAgent [candidate] (785.602 ms) : 0, 785602
GlobalTracer [baseline] (293.496 ms) : 0, 293496
GlobalTracer [candidate] (295.709 ms) : 0, 295709
AppSec [baseline] (47.077 ms) : 0, 47077
AppSec [candidate] (47.41 ms) : 0, 47410
IAST [baseline] (27.357 ms) : 0, 27357
IAST [candidate] (29.166 ms) : 0, 29166
Remote Config [baseline] (613.504 µs) : 0, 614
Remote Config [candidate] (649.233 µs) : 0, 649
Telemetry [baseline] (6.88 ms) : 0, 6880
Telemetry [candidate] (6.946 ms) : 0, 6946
section profiling
BytebuddyAgent [baseline] (665.099 ms) : 0, 665099
BytebuddyAgent [candidate] (668.453 ms) : 0, 668453
GlobalTracer [baseline] (387.126 ms) : 0, 387126
GlobalTracer [candidate] (389.104 ms) : 0, 389104
AppSec [baseline] (51.466 ms) : 0, 51466
AppSec [candidate] (52.226 ms) : 0, 52226
Remote Config [baseline] (743.391 µs) : 0, 743
Remote Config [candidate] (747.686 µs) : 0, 748
Telemetry [baseline] (7.419 ms) : 0, 7419
Telemetry [candidate] (7.494 ms) : 0, 7494
ProfilingAgent [baseline] (96.719 ms) : 0, 96719
ProfilingAgent [candidate] (97.118 ms) : 0, 97118
Profiling [baseline] (96.744 ms) : 0, 96744
Profiling [candidate] (97.144 ms) : 0, 97144
Startup time reports for insecure-bankgantt
title insecure-bank - global startup overhead: candidate=1.36.0-SNAPSHOT~44c8c2a5fa, baseline=1.36.0-SNAPSHOT~a19c6d5486
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.069 s) : 0, 1069391
Total [baseline] (8.549 s) : 0, 8548776
Agent [candidate] (1.062 s) : 0, 1062452
Total [candidate] (8.555 s) : 0, 8555471
section iast
Agent [baseline] (1.167 s) : 0, 1166654
Total [baseline] (9.021 s) : 0, 9021324
Agent [candidate] (1.177 s) : 0, 1177095
Total [candidate] (9.007 s) : 0, 9007081
section iast_HARDCODED_SECRET_DISABLED
Agent [baseline] (1.18 s) : 0, 1179707
Total [baseline] (9.036 s) : 0, 9036427
Agent [candidate] (1.172 s) : 0, 1171543
Total [candidate] (9.061 s) : 0, 9061060
section iast_TELEMETRY_OFF
Agent [baseline] (1.166 s) : 0, 1166425
Total [baseline] (9.027 s) : 0, 9027139
Agent [candidate] (1.167 s) : 0, 1166743
Total [candidate] (9.008 s) : 0, 9007606
gantt
title insecure-bank - break down per module: candidate=1.36.0-SNAPSHOT~44c8c2a5fa, baseline=1.36.0-SNAPSHOT~a19c6d5486
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (670.034 ms) : 0, 670034
BytebuddyAgent [candidate] (665.296 ms) : 0, 665296
GlobalTracer [baseline] (305.799 ms) : 0, 305799
GlobalTracer [candidate] (304.237 ms) : 0, 304237
AppSec [baseline] (50.662 ms) : 0, 50662
AppSec [candidate] (50.275 ms) : 0, 50275
Remote Config [baseline] (693.366 µs) : 0, 693
Remote Config [candidate] (689.264 µs) : 0, 689
Telemetry [baseline] (7.637 ms) : 0, 7637
Telemetry [candidate] (7.596 ms) : 0, 7596
section iast
BytebuddyAgent [baseline] (778.993 ms) : 0, 778993
BytebuddyAgent [candidate] (785.675 ms) : 0, 785675
GlobalTracer [baseline] (293.339 ms) : 0, 293339
GlobalTracer [candidate] (295.279 ms) : 0, 295279
AppSec [baseline] (47.067 ms) : 0, 47067
AppSec [candidate] (47.476 ms) : 0, 47476
IAST [baseline] (26.501 ms) : 0, 26501
IAST [candidate] (26.044 ms) : 0, 26044
Remote Config [baseline] (586.04 µs) : 0, 586
Remote Config [candidate] (659.412 µs) : 0, 659
Telemetry [baseline] (6.926 ms) : 0, 6926
Telemetry [candidate] (8.594 ms) : 0, 8594
section iast_HARDCODED_SECRET_DISABLED
BytebuddyAgent [baseline] (786.181 ms) : 0, 786181
BytebuddyAgent [candidate] (780.78 ms) : 0, 780780
GlobalTracer [baseline] (296.551 ms) : 0, 296551
GlobalTracer [candidate] (293.492 ms) : 0, 293492
AppSec [baseline] (47.668 ms) : 0, 47668
AppSec [candidate] (47.28 ms) : 0, 47280
IAST [baseline] (28.191 ms) : 0, 28191
IAST [candidate] (29.093 ms) : 0, 29093
Remote Config [baseline] (656.024 µs) : 0, 656
Remote Config [candidate] (620.265 µs) : 0, 620
Telemetry [baseline] (7.034 ms) : 0, 7034
Telemetry [candidate] (6.939 ms) : 0, 6939
section iast_TELEMETRY_OFF
BytebuddyAgent [baseline] (777.845 ms) : 0, 777845
BytebuddyAgent [candidate] (776.82 ms) : 0, 776820
GlobalTracer [baseline] (294.166 ms) : 0, 294166
GlobalTracer [candidate] (294.173 ms) : 0, 294173
AppSec [baseline] (47.151 ms) : 0, 47151
AppSec [candidate] (47.147 ms) : 0, 47147
IAST [baseline] (24.951 ms) : 0, 24951
IAST [candidate] (26.3 ms) : 0, 26300
Remote Config [baseline] (580.288 µs) : 0, 580
Remote Config [candidate] (598.639 µs) : 0, 599
Telemetry [baseline] (8.393 ms) : 0, 8393
Telemetry [candidate] (8.371 ms) : 0, 8371
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.36.0-SNAPSHOT~44c8c2a5fa, baseline=1.36.0-SNAPSHOT~a19c6d5486
dateFormat X
axisFormat %s
section baseline
no_agent (1.351 ms) : 1332, 1370
. : milestone, 1351,
appsec (1.748 ms) : 1724, 1771
. : milestone, 1748,
appsec_no_iast (1.718 ms) : 1693, 1743
. : milestone, 1718,
iast (1.475 ms) : 1453, 1498
. : milestone, 1475,
profiling (1.479 ms) : 1454, 1504
. : milestone, 1479,
tracing (1.47 ms) : 1446, 1493
. : milestone, 1470,
section candidate
no_agent (1.331 ms) : 1312, 1350
. : milestone, 1331,
appsec (1.746 ms) : 1723, 1770
. : milestone, 1746,
appsec_no_iast (1.738 ms) : 1714, 1763
. : milestone, 1738,
iast (1.484 ms) : 1461, 1507
. : milestone, 1484,
profiling (1.477 ms) : 1453, 1502
. : milestone, 1477,
tracing (1.46 ms) : 1436, 1485
. : milestone, 1460,
Request duration reports for insecure-bankgantt
title insecure-bank - request duration [CI 0.99] : candidate=1.36.0-SNAPSHOT~44c8c2a5fa, baseline=1.36.0-SNAPSHOT~a19c6d5486
dateFormat X
axisFormat %s
section baseline
no_agent (372.066 µs) : 351, 393
. : milestone, 372,
iast (492.036 µs) : 470, 514
. : milestone, 492,
iast_FULL (554.373 µs) : 533, 576
. : milestone, 554,
iast_GLOBAL (508.663 µs) : 488, 530
. : milestone, 509,
iast_HARDCODED_SECRET_DISABLED (489.252 µs) : 468, 511
. : milestone, 489,
iast_INACTIVE (461.766 µs) : 440, 484
. : milestone, 462,
iast_TELEMETRY_OFF (481.271 µs) : 459, 503
. : milestone, 481,
tracing (448.992 µs) : 428, 470
. : milestone, 449,
section candidate
no_agent (372.98 µs) : 353, 392
. : milestone, 373,
iast (489.975 µs) : 469, 511
. : milestone, 490,
iast_FULL (555.121 µs) : 532, 578
. : milestone, 555,
iast_GLOBAL (519.481 µs) : 496, 543
. : milestone, 519,
iast_HARDCODED_SECRET_DISABLED (490.763 µs) : 469, 512
. : milestone, 491,
iast_INACTIVE (460.895 µs) : 439, 483
. : milestone, 461,
iast_TELEMETRY_OFF (479.851 µs) : 458, 502
. : milestone, 480,
tracing (446.616 µs) : 426, 467
. : milestone, 447,
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.36.0-SNAPSHOT~44c8c2a5fa, baseline=1.36.0-SNAPSHOT~a19c6d5486
dateFormat X
axisFormat %s
section baseline
no_agent (1.465 ms) : 1454, 1477
. : milestone, 1465,
appsec (2.217 ms) : 2183, 2251
. : milestone, 2217,
iast (1.969 ms) : 1927, 2010
. : milestone, 1969,
iast_GLOBAL (2.016 ms) : 1974, 2058
. : milestone, 2016,
profiling (1.85 ms) : 1815, 1884
. : milestone, 1850,
tracing (1.834 ms) : 1802, 1866
. : milestone, 1834,
section candidate
no_agent (1.462 ms) : 1451, 1474
. : milestone, 1462,
appsec (2.194 ms) : 2161, 2228
. : milestone, 2194,
iast (1.971 ms) : 1930, 2013
. : milestone, 1971,
iast_GLOBAL (2.005 ms) : 1964, 2046
. : milestone, 2005,
profiling (1.857 ms) : 1822, 1891
. : milestone, 1857,
tracing (1.838 ms) : 1805, 1870
. : milestone, 1838,
Execution time for biojavagantt
title biojava - execution time [CI 0.99] : candidate=1.36.0-SNAPSHOT~44c8c2a5fa, baseline=1.36.0-SNAPSHOT~a19c6d5486
dateFormat X
axisFormat %s
section baseline
no_agent (15.344 s) : 15344000, 15344000
. : milestone, 15344000,
appsec (15.076 s) : 15076000, 15076000
. : milestone, 15076000,
iast (18.651 s) : 18651000, 18651000
. : milestone, 18651000,
iast_GLOBAL (17.795 s) : 17795000, 17795000
. : milestone, 17795000,
profiling (15.196 s) : 15196000, 15196000
. : milestone, 15196000,
tracing (15.017 s) : 15017000, 15017000
. : milestone, 15017000,
section candidate
no_agent (15.015 s) : 15015000, 15015000
. : milestone, 15015000,
appsec (15.21 s) : 15210000, 15210000
. : milestone, 15210000,
iast (18.912 s) : 18912000, 18912000
. : milestone, 18912000,
iast_GLOBAL (18.081 s) : 18081000, 18081000
. : milestone, 18081000,
profiling (14.973 s) : 14973000, 14973000
. : milestone, 14973000,
tracing (14.913 s) : 14913000, 14913000
. : milestone, 14913000,
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
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
Adds
LogCollector.EXCLUDE_TELEMETRY
marker to logs inMethodHandles
class.As the result, these logs will not be reported to telemetry.
Motivation
By default every log that contains a throwable is reported to telemetry regardless of its log level.
The
MethodHandles
class contains some debug logs that log errors that are expected to occur in some envs, depending on dependency versions that the traced app is using (example: in one version of a testing framework there is a field named "a", while in a later version of the same framework the same field is named "b" - attempting to get one of these will always fail and it will not be an error, which is why it is logged with debug level).These logs are there only in case a specific issue needs to be investigated, they are not needed in telemetry.
Jira ticket: SDTEST-461