-
Notifications
You must be signed in to change notification settings - Fork 323
Add single InstrumenterModule for group of instrumentation #10320
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
BenchmarksStartupParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 56 metrics, 9 unstable metrics. Startup time reports for petclinicgantt
title petclinic - global startup overhead: candidate=1.59.0-SNAPSHOT~ae2c6cd137, baseline=1.59.0-SNAPSHOT~8ef6e7ab7b
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.085 s) : 0, 1084768
Total [baseline] (10.743 s) : 0, 10742519
Agent [candidate] (1.086 s) : 0, 1085550
Total [candidate] (10.825 s) : 0, 10825079
section appsec
Agent [baseline] (1.274 s) : 0, 1274227
Total [baseline] (10.986 s) : 0, 10986130
Agent [candidate] (1.268 s) : 0, 1267855
Total [candidate] (10.898 s) : 0, 10897540
section iast
Agent [baseline] (1.233 s) : 0, 1233447
Total [baseline] (11.264 s) : 0, 11263713
Agent [candidate] (1.224 s) : 0, 1224034
Total [candidate] (11.079 s) : 0, 11078874
section profiling
Agent [baseline] (1.22 s) : 0, 1219579
Total [baseline] (10.985 s) : 0, 10985453
Agent [candidate] (1.204 s) : 0, 1203902
Total [candidate] (10.913 s) : 0, 10913396
gantt
title petclinic - break down per module: candidate=1.59.0-SNAPSHOT~ae2c6cd137, baseline=1.59.0-SNAPSHOT~8ef6e7ab7b
dateFormat X
axisFormat %s
section tracing
crashtracking [baseline] (1.182 ms) : 0, 1182
crashtracking [candidate] (1.186 ms) : 0, 1186
BytebuddyAgent [baseline] (650.999 ms) : 0, 650999
BytebuddyAgent [candidate] (651.447 ms) : 0, 651447
GlobalTracer [baseline] (282.528 ms) : 0, 282528
GlobalTracer [candidate] (283.257 ms) : 0, 283257
AppSec [baseline] (32.749 ms) : 0, 32749
AppSec [candidate] (32.533 ms) : 0, 32533
Debugger [baseline] (68.446 ms) : 0, 68446
Debugger [candidate] (68.18 ms) : 0, 68180
Remote Config [baseline] (619.116 µs) : 0, 619
Remote Config [candidate] (635.621 µs) : 0, 636
Telemetry [baseline] (8.959 ms) : 0, 8959
Telemetry [candidate] (8.882 ms) : 0, 8882
Flare Poller [baseline] (3.724 ms) : 0, 3724
Flare Poller [candidate] (3.798 ms) : 0, 3798
section appsec
crashtracking [baseline] (1.193 ms) : 0, 1193
crashtracking [candidate] (1.18 ms) : 0, 1180
BytebuddyAgent [baseline] (696.62 ms) : 0, 696620
BytebuddyAgent [candidate] (692.989 ms) : 0, 692989
GlobalTracer [baseline] (259.952 ms) : 0, 259952
GlobalTracer [candidate] (258.338 ms) : 0, 258338
AppSec [baseline] (175.021 ms) : 0, 175021
AppSec [candidate] (173.697 ms) : 0, 173697
Debugger [baseline] (66.949 ms) : 0, 66949
Debugger [candidate] (67.641 ms) : 0, 67641
Remote Config [baseline] (747.928 µs) : 0, 748
Remote Config [candidate] (769.921 µs) : 0, 770
Telemetry [baseline] (9.457 ms) : 0, 9457
Telemetry [candidate] (9.449 ms) : 0, 9449
Flare Poller [baseline] (3.868 ms) : 0, 3868
Flare Poller [candidate] (3.716 ms) : 0, 3716
IAST [baseline] (24.792 ms) : 0, 24792
IAST [candidate] (24.581 ms) : 0, 24581
section iast
crashtracking [baseline] (1.188 ms) : 0, 1188
crashtracking [candidate] (1.197 ms) : 0, 1197
BytebuddyAgent [baseline] (797.546 ms) : 0, 797546
BytebuddyAgent [candidate] (791.449 ms) : 0, 791449
GlobalTracer [baseline] (257.91 ms) : 0, 257910
GlobalTracer [candidate] (255.818 ms) : 0, 255818
AppSec [baseline] (34.83 ms) : 0, 34830
AppSec [candidate] (32.898 ms) : 0, 32898
Debugger [baseline] (66.159 ms) : 0, 66159
Debugger [candidate] (67.705 ms) : 0, 67705
Remote Config [baseline] (569.503 µs) : 0, 570
Remote Config [candidate] (566.634 µs) : 0, 567
Telemetry [baseline] (8.565 ms) : 0, 8565
Telemetry [candidate] (8.502 ms) : 0, 8502
Flare Poller [baseline] (3.614 ms) : 0, 3614
Flare Poller [candidate] (3.508 ms) : 0, 3508
IAST [baseline] (27.474 ms) : 0, 27474
IAST [candidate] (26.926 ms) : 0, 26926
section profiling
crashtracking [baseline] (1.228 ms) : 0, 1228
crashtracking [candidate] (1.213 ms) : 0, 1213
BytebuddyAgent [baseline] (711.378 ms) : 0, 711378
BytebuddyAgent [candidate] (701.023 ms) : 0, 701023
GlobalTracer [baseline] (223.214 ms) : 0, 223214
GlobalTracer [candidate] (220.773 ms) : 0, 220773
AppSec [baseline] (32.953 ms) : 0, 32953
AppSec [candidate] (32.223 ms) : 0, 32223
Debugger [baseline] (68.821 ms) : 0, 68821
Debugger [candidate] (68.141 ms) : 0, 68141
Remote Config [baseline] (650.51 µs) : 0, 651
Remote Config [candidate] (607.568 µs) : 0, 608
Telemetry [baseline] (8.863 ms) : 0, 8863
Telemetry [candidate] (8.819 ms) : 0, 8819
Flare Poller [baseline] (3.707 ms) : 0, 3707
Flare Poller [candidate] (3.736 ms) : 0, 3736
ProfilingAgent [baseline] (98.333 ms) : 0, 98333
ProfilingAgent [candidate] (97.67 ms) : 0, 97670
Profiling [baseline] (98.924 ms) : 0, 98924
Profiling [candidate] (98.243 ms) : 0, 98243
Startup time reports for insecure-bankgantt
title insecure-bank - global startup overhead: candidate=1.59.0-SNAPSHOT~ae2c6cd137, baseline=1.59.0-SNAPSHOT~8ef6e7ab7b
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.081 s) : 0, 1081329
Total [baseline] (8.718 s) : 0, 8718115
Agent [candidate] (1.084 s) : 0, 1084294
Total [candidate] (8.756 s) : 0, 8756318
section iast
Agent [baseline] (1.223 s) : 0, 1223469
Total [baseline] (9.284 s) : 0, 9284313
Agent [candidate] (1.223 s) : 0, 1222782
Total [candidate] (9.328 s) : 0, 9328142
gantt
title insecure-bank - break down per module: candidate=1.59.0-SNAPSHOT~ae2c6cd137, baseline=1.59.0-SNAPSHOT~8ef6e7ab7b
dateFormat X
axisFormat %s
section tracing
crashtracking [baseline] (1.187 ms) : 0, 1187
crashtracking [candidate] (1.181 ms) : 0, 1181
BytebuddyAgent [baseline] (649.375 ms) : 0, 649375
BytebuddyAgent [candidate] (651.066 ms) : 0, 651066
GlobalTracer [baseline] (282.141 ms) : 0, 282141
GlobalTracer [candidate] (282.924 ms) : 0, 282924
AppSec [baseline] (32.556 ms) : 0, 32556
AppSec [candidate] (32.742 ms) : 0, 32742
Debugger [baseline] (67.091 ms) : 0, 67091
Debugger [candidate] (67.379 ms) : 0, 67379
Remote Config [baseline] (628.435 µs) : 0, 628
Remote Config [candidate] (633.685 µs) : 0, 634
Telemetry [baseline] (9.045 ms) : 0, 9045
Telemetry [candidate] (9.026 ms) : 0, 9026
Flare Poller [baseline] (3.734 ms) : 0, 3734
Flare Poller [candidate] (3.784 ms) : 0, 3784
section iast
crashtracking [baseline] (1.177 ms) : 0, 1177
crashtracking [candidate] (1.181 ms) : 0, 1181
BytebuddyAgent [baseline] (791.333 ms) : 0, 791333
BytebuddyAgent [candidate] (791.327 ms) : 0, 791327
GlobalTracer [baseline] (255.848 ms) : 0, 255848
GlobalTracer [candidate] (255.523 ms) : 0, 255523
AppSec [baseline] (32.794 ms) : 0, 32794
AppSec [candidate] (33.798 ms) : 0, 33798
Debugger [baseline] (67.122 ms) : 0, 67122
Debugger [candidate] (65.96 ms) : 0, 65960
Remote Config [baseline] (579.515 µs) : 0, 580
Remote Config [candidate] (604.436 µs) : 0, 604
Telemetry [baseline] (8.485 ms) : 0, 8485
Telemetry [candidate] (8.532 ms) : 0, 8532
Flare Poller [baseline] (3.598 ms) : 0, 3598
Flare Poller [candidate] (3.482 ms) : 0, 3482
IAST [baseline] (27.126 ms) : 0, 27126
IAST [candidate] (26.965 ms) : 0, 26965
LoadParameters
See matching parameters
SummaryFound 1 performance improvements and 3 performance regressions! Performance is the same for 16 metrics, 16 unstable metrics.
Request duration reports for insecure-bankgantt
title insecure-bank - request duration [CI 0.99] : candidate=1.59.0-SNAPSHOT~ae2c6cd137, baseline=1.59.0-SNAPSHOT~8ef6e7ab7b
dateFormat X
axisFormat %s
section baseline
no_agent (1.179 ms) : 1167, 1190
. : milestone, 1179,
iast (3.372 ms) : 3331, 3413
. : milestone, 3372,
iast_FULL (5.546 ms) : 5492, 5599
. : milestone, 5546,
iast_GLOBAL (3.64 ms) : 3583, 3697
. : milestone, 3640,
profiling (1.896 ms) : 1881, 1912
. : milestone, 1896,
tracing (1.794 ms) : 1778, 1809
. : milestone, 1794,
section candidate
no_agent (1.187 ms) : 1176, 1199
. : milestone, 1187,
iast (3.247 ms) : 3203, 3292
. : milestone, 3247,
iast_FULL (5.791 ms) : 5733, 5849
. : milestone, 5791,
iast_GLOBAL (3.659 ms) : 3598, 3721
. : milestone, 3659,
profiling (2.019 ms) : 2001, 2036
. : milestone, 2019,
tracing (1.731 ms) : 1716, 1745
. : milestone, 1731,
Request duration reports for petclinicgantt
title petclinic - request duration [CI 0.99] : candidate=1.59.0-SNAPSHOT~ae2c6cd137, baseline=1.59.0-SNAPSHOT~8ef6e7ab7b
dateFormat X
axisFormat %s
section baseline
no_agent (19.431 ms) : 19229, 19633
. : milestone, 19431,
appsec (18.767 ms) : 18575, 18958
. : milestone, 18767,
code_origins (17.785 ms) : 17608, 17962
. : milestone, 17785,
iast (17.687 ms) : 17508, 17865
. : milestone, 17687,
profiling (19.6 ms) : 19405, 19795
. : milestone, 19600,
tracing (17.64 ms) : 17464, 17816
. : milestone, 17640,
section candidate
no_agent (17.892 ms) : 17709, 18075
. : milestone, 17892,
appsec (19.574 ms) : 19371, 19776
. : milestone, 19574,
code_origins (17.482 ms) : 17307, 17658
. : milestone, 17482,
iast (17.534 ms) : 17359, 17709
. : milestone, 17534,
profiling (19.256 ms) : 19063, 19448
. : milestone, 19256,
tracing (17.606 ms) : 17433, 17779
. : milestone, 17606,
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.59.0-SNAPSHOT~ae2c6cd137, baseline=1.59.0-SNAPSHOT~8ef6e7ab7b
dateFormat X
axisFormat %s
section baseline
no_agent (1.47 ms) : 1459, 1482
. : milestone, 1470,
appsec (3.641 ms) : 3426, 3856
. : milestone, 3641,
iast (2.201 ms) : 2136, 2265
. : milestone, 2201,
iast_GLOBAL (2.246 ms) : 2181, 2311
. : milestone, 2246,
profiling (2.079 ms) : 2025, 2133
. : milestone, 2079,
tracing (2.039 ms) : 1988, 2090
. : milestone, 2039,
section candidate
no_agent (1.473 ms) : 1462, 1485
. : milestone, 1473,
appsec (3.675 ms) : 3459, 3891
. : milestone, 3675,
iast (2.2 ms) : 2136, 2265
. : milestone, 2200,
iast_GLOBAL (2.248 ms) : 2183, 2314
. : milestone, 2248,
profiling (2.084 ms) : 2029, 2138
. : milestone, 2084,
tracing (2.04 ms) : 1989, 2091
. : milestone, 2040,
Execution time for biojavagantt
title biojava - execution time [CI 0.99] : candidate=1.59.0-SNAPSHOT~ae2c6cd137, baseline=1.59.0-SNAPSHOT~8ef6e7ab7b
dateFormat X
axisFormat %s
section baseline
no_agent (14.75 s) : 14750000, 14750000
. : milestone, 14750000,
appsec (14.674 s) : 14674000, 14674000
. : milestone, 14674000,
iast (17.947 s) : 17947000, 17947000
. : milestone, 17947000,
iast_GLOBAL (17.894 s) : 17894000, 17894000
. : milestone, 17894000,
profiling (15.365 s) : 15365000, 15365000
. : milestone, 15365000,
tracing (14.579 s) : 14579000, 14579000
. : milestone, 14579000,
section candidate
no_agent (14.961 s) : 14961000, 14961000
. : milestone, 14961000,
appsec (14.741 s) : 14741000, 14741000
. : milestone, 14741000,
iast (18.106 s) : 18106000, 18106000
. : milestone, 18106000,
iast_GLOBAL (17.693 s) : 17693000, 17693000
. : milestone, 17693000,
profiling (14.704 s) : 14704000, 14704000
. : milestone, 14704000,
tracing (14.672 s) : 14672000, 14672000
. : milestone, 14672000,
|
PerfectSlayer
left a comment
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.
Looking good. It looks to remove some duplication for helper declaration but remove some fine grained control over instrumentation enablement.
It still feels to go in the right direction 👍
It can still be done in the |
What Does This Do
Migrate legacy instrumentations by separating InstrumenterModule from single Instrumenter
Motivation
Additional Notes
Contributor Checklist
type:and (comp:orinst:) labels in addition to any useful labelsclose,fixor any linking keywords when referencing an issue.Use
solvesinstead, and assign the PR milestone to the issueJira ticket: [PROJ-IDENT]