-
Notifications
You must be signed in to change notification settings - Fork 312
Activate client stats by default for java 17 services #9463
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
base: master
Are you sure you want to change the base?
Conversation
Hi! 👋 Thanks for your pull request! 🎉 To help us review it, please make sure to:
If you need help, please check our contributing guidelines. |
🎯 Code Coverage 🔗 Commit SHA: 9374c82 | Docs | Was this helpful? Give us feedback! |
BenchmarksStartupParameters
See matching parameters
SummaryFound 0 performance improvements and 6 performance regressions! Performance is the same for 39 metrics, 14 unstable metrics.
Startup time reports for insecure-bankgantt
title insecure-bank - global startup overhead: candidate=1.54.0-SNAPSHOT~9374c8254f, baseline=1.54.0-SNAPSHOT~35a825a3f4
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.046 s) : 0, 1046232
Total [baseline] (8.618 s) : 0, 8617870
Agent [candidate] (1.059 s) : 0, 1059288
Total [candidate] (8.651 s) : 0, 8651075
section iast
Agent [baseline] (1.177 s) : 0, 1177097
Total [baseline] (9.356 s) : 0, 9355798
Agent [candidate] (1.196 s) : 0, 1195711
Total [candidate] (9.33 s) : 0, 9330236
gantt
title insecure-bank - break down per module: candidate=1.54.0-SNAPSHOT~9374c8254f, baseline=1.54.0-SNAPSHOT~35a825a3f4
dateFormat X
axisFormat %s
section tracing
crashtracking [baseline] (1.45 ms) : 0, 1450
crashtracking [candidate] (1.463 ms) : 0, 1463
BytebuddyAgent [baseline] (732.393 ms) : 0, 732393
BytebuddyAgent [candidate] (732.549 ms) : 0, 732549
GlobalTracer [baseline] (240.852 ms) : 0, 240852
GlobalTracer [candidate] (250.873 ms) : 0, 250873
AppSec [baseline] (30.111 ms) : 0, 30111
AppSec [candidate] (30.315 ms) : 0, 30315
Debugger [baseline] (6.141 ms) : 0, 6141
Debugger [candidate] (6.048 ms) : 0, 6048
Remote Config [baseline] (668.123 µs) : 0, 668
Remote Config [candidate] (693.163 µs) : 0, 693
Telemetry [baseline] (13.643 ms) : 0, 13643
Telemetry [candidate] (16.344 ms) : 0, 16344
section iast
crashtracking [baseline] (1.462 ms) : 0, 1462
crashtracking [candidate] (1.476 ms) : 0, 1476
BytebuddyAgent [baseline] (850.811 ms) : 0, 850811
BytebuddyAgent [candidate] (858.261 ms) : 0, 858261
GlobalTracer [baseline] (232.655 ms) : 0, 232655
GlobalTracer [candidate] (244.762 ms) : 0, 244762
AppSec [baseline] (26.698 ms) : 0, 26698
AppSec [candidate] (26.322 ms) : 0, 26322
Debugger [baseline] (6.556 ms) : 0, 6556
Debugger [candidate] (5.78 ms) : 0, 5780
Remote Config [baseline] (601.938 µs) : 0, 602
Remote Config [candidate] (594.387 µs) : 0, 594
Telemetry [baseline] (8.107 ms) : 0, 8107
Telemetry [candidate] (8.262 ms) : 0, 8262
IAST [baseline] (29.213 ms) : 0, 29213
IAST [candidate] (29.196 ms) : 0, 29196
Startup time reports for petclinicgantt
title petclinic - global startup overhead: candidate=1.54.0-SNAPSHOT~9374c8254f, baseline=1.54.0-SNAPSHOT~35a825a3f4
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.048 s) : 0, 1047682
Total [baseline] (10.614 s) : 0, 10613692
Agent [candidate] (1.065 s) : 0, 1065207
Total [candidate] (10.758 s) : 0, 10757527
section appsec
Agent [baseline] (1.223 s) : 0, 1223127
Total [baseline] (10.786 s) : 0, 10785546
Agent [candidate] (1.235 s) : 0, 1234689
Total [candidate] (10.901 s) : 0, 10900541
section iast
Agent [baseline] (1.187 s) : 0, 1187474
Total [baseline] (10.972 s) : 0, 10972233
Agent [candidate] (1.188 s) : 0, 1188012
Total [candidate] (11.055 s) : 0, 11055010
section profiling
Agent [baseline] (1.197 s) : 0, 1197297
Total [baseline] (10.917 s) : 0, 10916833
Agent [candidate] (1.21 s) : 0, 1209852
Total [candidate] (10.94 s) : 0, 10940128
gantt
title petclinic - break down per module: candidate=1.54.0-SNAPSHOT~9374c8254f, baseline=1.54.0-SNAPSHOT~35a825a3f4
dateFormat X
axisFormat %s
section tracing
crashtracking [baseline] (1.446 ms) : 0, 1446
crashtracking [candidate] (1.443 ms) : 0, 1443
BytebuddyAgent [baseline] (733.842 ms) : 0, 733842
BytebuddyAgent [candidate] (734.656 ms) : 0, 734656
GlobalTracer [baseline] (241.309 ms) : 0, 241309
GlobalTracer [candidate] (253.979 ms) : 0, 253979
AppSec [baseline] (30.235 ms) : 0, 30235
AppSec [candidate] (30.759 ms) : 0, 30759
Debugger [baseline] (6.087 ms) : 0, 6087
Debugger [candidate] (6.127 ms) : 0, 6127
Remote Config [baseline] (684.46 µs) : 0, 684
Remote Config [candidate] (694.543 µs) : 0, 695
Telemetry [baseline] (12.922 ms) : 0, 12922
Telemetry [candidate] (16.39 ms) : 0, 16390
section appsec
crashtracking [baseline] (1.455 ms) : 0, 1455
crashtracking [candidate] (1.443 ms) : 0, 1443
BytebuddyAgent [baseline] (756.633 ms) : 0, 756633
BytebuddyAgent [candidate] (758.222 ms) : 0, 758222
GlobalTracer [baseline] (234.116 ms) : 0, 234116
GlobalTracer [candidate] (244.717 ms) : 0, 244717
IAST [baseline] (23.48 ms) : 0, 23480
IAST [candidate] (23.741 ms) : 0, 23741
AppSec [baseline] (169.053 ms) : 0, 169053
AppSec [candidate] (170.578 ms) : 0, 170578
Debugger [baseline] (7.42 ms) : 0, 7420
Debugger [candidate] (5.76 ms) : 0, 5760
Remote Config [baseline] (622.088 µs) : 0, 622
Remote Config [candidate] (642.635 µs) : 0, 643
Telemetry [baseline] (9.233 ms) : 0, 9233
Telemetry [candidate] (8.357 ms) : 0, 8357
section iast
crashtracking [baseline] (1.461 ms) : 0, 1461
crashtracking [candidate] (1.446 ms) : 0, 1446
BytebuddyAgent [baseline] (859.273 ms) : 0, 859273
BytebuddyAgent [candidate] (852.468 ms) : 0, 852468
GlobalTracer [baseline] (232.756 ms) : 0, 232756
GlobalTracer [candidate] (245.498 ms) : 0, 245498
IAST [baseline] (32.735 ms) : 0, 32735
IAST [candidate] (28.05 ms) : 0, 28050
AppSec [baseline] (24.555 ms) : 0, 24555
AppSec [candidate] (25.156 ms) : 0, 25156
Debugger [baseline] (6.573 ms) : 0, 6573
Debugger [candidate] (5.685 ms) : 0, 5685
Remote Config [baseline] (611.489 µs) : 0, 611
Remote Config [candidate] (586.919 µs) : 0, 587
Telemetry [baseline] (8.282 ms) : 0, 8282
Telemetry [candidate] (7.992 ms) : 0, 7992
section profiling
crashtracking [baseline] (1.437 ms) : 0, 1437
crashtracking [candidate] (1.428 ms) : 0, 1428
BytebuddyAgent [baseline] (762.426 ms) : 0, 762426
BytebuddyAgent [candidate] (762.964 ms) : 0, 762964
GlobalTracer [baseline] (221.918 ms) : 0, 221918
GlobalTracer [candidate] (231.732 ms) : 0, 231732
AppSec [baseline] (31.322 ms) : 0, 31322
AppSec [candidate] (31.035 ms) : 0, 31035
Debugger [baseline] (6.283 ms) : 0, 6283
Debugger [candidate] (6.372 ms) : 0, 6372
Remote Config [baseline] (716.702 µs) : 0, 717
Remote Config [candidate] (680.361 µs) : 0, 680
Telemetry [baseline] (14.851 ms) : 0, 14851
Telemetry [candidate] (15.888 ms) : 0, 15888
ProfilingAgent [baseline] (107.944 ms) : 0, 107944
ProfilingAgent [candidate] (109.19 ms) : 0, 109190
Profiling [baseline] (108.601 ms) : 0, 108601
Profiling [candidate] (109.81 ms) : 0, 109810
LoadParameters
See matching parameters
SummaryFound 2 performance improvements and 1 performance regressions! Performance is the same for 9 metrics, 12 unstable metrics.
Request duration reports for insecure-bankgantt
title insecure-bank - request duration [CI 0.99] : candidate=1.54.0-SNAPSHOT~9374c8254f, baseline=1.54.0-SNAPSHOT~35a825a3f4
dateFormat X
axisFormat %s
section baseline
no_agent (4.563 ms) : 4511, 4615
. : milestone, 4563,
iast (9.065 ms) : 8920, 9209
. : milestone, 9065,
iast_FULL (13.866 ms) : 13589, 14144
. : milestone, 13866,
iast_GLOBAL (10.642 ms) : 10455, 10830
. : milestone, 10642,
profiling (8.764 ms) : 8627, 8902
. : milestone, 8764,
tracing (7.538 ms) : 7432, 7644
. : milestone, 7538,
section candidate
no_agent (4.389 ms) : 4339, 4438
. : milestone, 4389,
iast (9.109 ms) : 8963, 9254
. : milestone, 9109,
iast_FULL (14.079 ms) : 13802, 14355
. : milestone, 14079,
iast_GLOBAL (10.452 ms) : 10270, 10634
. : milestone, 10452,
profiling (8.676 ms) : 8534, 8819
. : milestone, 8676,
tracing (7.529 ms) : 7424, 7634
. : milestone, 7529,
Request duration reports for petclinicgantt
title petclinic - request duration [CI 0.99] : candidate=1.54.0-SNAPSHOT~9374c8254f, baseline=1.54.0-SNAPSHOT~35a825a3f4
dateFormat X
axisFormat %s
section baseline
no_agent (38.146 ms) : 37825, 38468
. : milestone, 38146,
appsec (47.494 ms) : 47080, 47908
. : milestone, 47494,
code_origins (44.701 ms) : 44304, 45099
. : milestone, 44701,
iast (44.789 ms) : 44382, 45195
. : milestone, 44789,
profiling (49.409 ms) : 48948, 49869
. : milestone, 49409,
tracing (43.233 ms) : 42866, 43600
. : milestone, 43233,
section candidate
no_agent (35.34 ms) : 35051, 35629
. : milestone, 35340,
appsec (47.966 ms) : 47546, 48385
. : milestone, 47966,
code_origins (46.129 ms) : 45726, 46532
. : milestone, 46129,
iast (44.748 ms) : 44351, 45145
. : milestone, 44748,
profiling (49.07 ms) : 48612, 49528
. : milestone, 49070,
tracing (43.625 ms) : 43266, 43984
. : milestone, 43625,
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 biojavagantt
title biojava - execution time [CI 0.99] : candidate=1.54.0-SNAPSHOT~9374c8254f, baseline=1.54.0-SNAPSHOT~35a825a3f4
dateFormat X
axisFormat %s
section baseline
no_agent (14.959 s) : 14959000, 14959000
. : milestone, 14959000,
appsec (14.894 s) : 14894000, 14894000
. : milestone, 14894000,
iast (18.405 s) : 18405000, 18405000
. : milestone, 18405000,
iast_GLOBAL (18.271 s) : 18271000, 18271000
. : milestone, 18271000,
profiling (15.969 s) : 15969000, 15969000
. : milestone, 15969000,
tracing (14.971 s) : 14971000, 14971000
. : milestone, 14971000,
section candidate
no_agent (15.718 s) : 15718000, 15718000
. : milestone, 15718000,
appsec (15.053 s) : 15053000, 15053000
. : milestone, 15053000,
iast (18.807 s) : 18807000, 18807000
. : milestone, 18807000,
iast_GLOBAL (17.909 s) : 17909000, 17909000
. : milestone, 17909000,
profiling (15.4 s) : 15400000, 15400000
. : milestone, 15400000,
tracing (15.265 s) : 15265000, 15265000
. : milestone, 15265000,
Execution time for tomcatgantt
title tomcat - execution time [CI 0.99] : candidate=1.54.0-SNAPSHOT~9374c8254f, baseline=1.54.0-SNAPSHOT~35a825a3f4
dateFormat X
axisFormat %s
section baseline
no_agent (1.477 ms) : 1465, 1488
. : milestone, 1477,
appsec (3.673 ms) : 3457, 3888
. : milestone, 3673,
iast (2.199 ms) : 2136, 2261
. : milestone, 2199,
iast_GLOBAL (2.238 ms) : 2175, 2300
. : milestone, 2238,
profiling (2.055 ms) : 2005, 2106
. : milestone, 2055,
tracing (2.03 ms) : 1982, 2079
. : milestone, 2030,
section candidate
no_agent (1.479 ms) : 1467, 1490
. : milestone, 1479,
appsec (3.663 ms) : 3447, 3878
. : milestone, 3663,
iast (2.211 ms) : 2149, 2274
. : milestone, 2211,
iast_GLOBAL (2.243 ms) : 2180, 2306
. : milestone, 2243,
profiling (2.075 ms) : 2023, 2128
. : milestone, 2075,
tracing (2.022 ms) : 1973, 2070
. : milestone, 2022,
|
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.
🎯 suggestion: I would add a quick documentation about why Java Version filter is used (as an arbitrary rule for progressive rollout, not related to a Java feature)
@@ -1808,7 +1808,9 @@ private Config(final ConfigProvider configProvider, final InstrumenterConfig ins | |||
// Enable tracer computed trace metrics by default for Azure Functions | |||
tracerMetricsEnabled = | |||
configProvider.getBoolean( | |||
TRACE_STATS_COMPUTATION_ENABLED, azureFunctions, TRACER_METRICS_ENABLED); | |||
TRACE_STATS_COMPUTATION_ENABLED, | |||
azureFunctions || JavaVirtualMachine.isJavaVersion(17), |
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.
- Does it make sense to check exact version? what would happened on 17+ Javas?
- Does it make sense to add test for Java 17(+ ?)?
What Does This Do
Activates CSS by default for java 17 services
Motivation
Additional Notes
Contributor Checklist
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: [PROJ-IDENT]