-
Notifications
You must be signed in to change notification settings - Fork 293
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
IAST doesn't actively make remote calls, so never delay starting it #8294
Conversation
ce45369
to
818116f
Compare
BenchmarksStartupParameters
See matching parameters
SummaryFound 5 performance improvements and 0 performance regressions! Performance is the same for 50 metrics, 8 unstable metrics.
Startup time reports for petclinicgantt
title petclinic - global startup overhead: candidate=1.46.0-SNAPSHOT~818116f820, baseline=1.46.0-SNAPSHOT~e3793055b5
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.036 s) : 0, 1036400
Total [baseline] (10.452 s) : 0, 10452489
Agent [candidate] (1.041 s) : 0, 1040954
Total [candidate] (10.501 s) : 0, 10500833
section appsec
Agent [baseline] (1.178 s) : 0, 1177800
Total [baseline] (10.752 s) : 0, 10751848
Agent [candidate] (1.184 s) : 0, 1184112
Total [candidate] (10.796 s) : 0, 10796309
section iast
Agent [baseline] (1.175 s) : 0, 1175386
Total [baseline] (10.959 s) : 0, 10959365
Agent [candidate] (1.176 s) : 0, 1176098
Total [candidate] (10.981 s) : 0, 10980798
section profiling
Agent [baseline] (1.262 s) : 0, 1261578
Total [baseline] (10.866 s) : 0, 10865839
Agent [candidate] (1.26 s) : 0, 1259586
Total [candidate] (10.861 s) : 0, 10861461
gantt
title petclinic - break down per module: candidate=1.46.0-SNAPSHOT~818116f820, baseline=1.46.0-SNAPSHOT~e3793055b5
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (712.649 ms) : 0, 712649
BytebuddyAgent [candidate] (715.679 ms) : 0, 715679
GlobalTracer [baseline] (239.335 ms) : 0, 239335
GlobalTracer [candidate] (240.184 ms) : 0, 240184
AppSec [baseline] (55.682 ms) : 0, 55682
AppSec [candidate] (55.326 ms) : 0, 55326
Remote Config [baseline] (706.244 µs) : 0, 706
Remote Config [candidate] (721.599 µs) : 0, 722
Telemetry [baseline] (12.793 ms) : 0, 12793
Telemetry [candidate] (13.689 ms) : 0, 13689
section appsec
BytebuddyAgent [baseline] (730.704 ms) : 0, 730704
BytebuddyAgent [candidate] (734.205 ms) : 0, 734205
GlobalTracer [baseline] (236.847 ms) : 0, 236847
GlobalTracer [candidate] (238.028 ms) : 0, 238028
AppSec [baseline] (176.137 ms) : 0, 176137
AppSec [candidate] (176.717 ms) : 0, 176717
Remote Config [baseline] (645.75 µs) : 0, 646
Remote Config [candidate] (654.877 µs) : 0, 655
Telemetry [baseline] (8.182 ms) : 0, 8182
Telemetry [candidate] (8.331 ms) : 0, 8331
IAST [baseline] (20.012 ms) : 0, 20012
IAST [candidate] (21.643 ms) : 0, 21643
section iast
BytebuddyAgent [baseline] (838.14 ms) : 0, 838140
BytebuddyAgent [candidate] (839.303 ms) : 0, 839303
GlobalTracer [baseline] (232.709 ms) : 0, 232709
GlobalTracer [candidate] (232.462 ms) : 0, 232462
AppSec [baseline] (58.154 ms) : 0, 58154
AppSec [candidate] (54.504 ms) : 0, 54504
Remote Config [baseline] (656.042 µs) : 0, 656
Remote Config [candidate] (614.332 µs) : 0, 614
Telemetry [baseline] (8.732 ms) : 0, 8732
Telemetry [candidate] (8.73 ms) : 0, 8730
IAST [baseline] (21.601 ms) : 0, 21601
IAST [candidate] (24.973 ms) : 0, 24973
section profiling
ProfilingAgent [baseline] (97.11 ms) : 0, 97110
ProfilingAgent [candidate] (95.819 ms) : 0, 95819
BytebuddyAgent [baseline] (706.704 ms) : 0, 706704
BytebuddyAgent [candidate] (706.732 ms) : 0, 706732
GlobalTracer [baseline] (350.167 ms) : 0, 350167
GlobalTracer [candidate] (350.786 ms) : 0, 350786
AppSec [baseline] (55.383 ms) : 0, 55383
AppSec [candidate] (54.356 ms) : 0, 54356
Remote Config [baseline] (709.535 µs) : 0, 710
Remote Config [candidate] (696.976 µs) : 0, 697
Telemetry [baseline] (8.861 ms) : 0, 8861
Telemetry [candidate] (8.868 ms) : 0, 8868
Profiling [baseline] (97.133 ms) : 0, 97133
Profiling [candidate] (95.842 ms) : 0, 95842
Startup time reports for insecure-bankgantt
title insecure-bank - global startup overhead: candidate=1.46.0-SNAPSHOT~818116f820, baseline=1.46.0-SNAPSHOT~e3793055b5
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.038 s) : 0, 1038082
Total [baseline] (8.643 s) : 0, 8642570
Agent [candidate] (1.036 s) : 0, 1036090
Total [candidate] (8.663 s) : 0, 8662963
section iast
Agent [baseline] (1.171 s) : 0, 1171152
Total [baseline] (9.236 s) : 0, 9236097
Agent [candidate] (1.179 s) : 0, 1179281
Total [candidate] (9.272 s) : 0, 9272437
section iast_HARDCODED_SECRET_DISABLED
Agent [baseline] (1.166 s) : 0, 1166216
Total [baseline] (9.192 s) : 0, 9192085
Agent [candidate] (1.169 s) : 0, 1169141
Total [candidate] (9.203 s) : 0, 9203336
section iast_TELEMETRY_OFF
Agent [baseline] (1.163 s) : 0, 1162951
Total [baseline] (9.228 s) : 0, 9227714
Agent [candidate] (1.166 s) : 0, 1165575
Total [candidate] (9.233 s) : 0, 9232600
gantt
title insecure-bank - break down per module: candidate=1.46.0-SNAPSHOT~818116f820, baseline=1.46.0-SNAPSHOT~e3793055b5
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (713.734 ms) : 0, 713734
BytebuddyAgent [candidate] (712.95 ms) : 0, 712950
GlobalTracer [baseline] (240.918 ms) : 0, 240918
GlobalTracer [candidate] (239.295 ms) : 0, 239295
AppSec [baseline] (55.873 ms) : 0, 55873
AppSec [candidate] (55.1 ms) : 0, 55100
Remote Config [baseline] (749.06 µs) : 0, 749
Remote Config [candidate] (707.603 µs) : 0, 708
Telemetry [baseline] (11.561 ms) : 0, 11561
Telemetry [candidate] (12.8 ms) : 0, 12800
section iast
BytebuddyAgent [baseline] (835.828 ms) : 0, 835828
BytebuddyAgent [candidate] (841.558 ms) : 0, 841558
GlobalTracer [baseline] (231.275 ms) : 0, 231275
GlobalTracer [candidate] (232.638 ms) : 0, 232638
AppSec [baseline] (57.96 ms) : 0, 57960
AppSec [candidate] (54.197 ms) : 0, 54197
Remote Config [baseline] (660.374 µs) : 0, 660
Remote Config [candidate] (613.161 µs) : 0, 613
Telemetry [baseline] (8.695 ms) : 0, 8695
Telemetry [candidate] (8.897 ms) : 0, 8897
IAST [baseline] (21.396 ms) : 0, 21396
IAST [candidate] (25.81 ms) : 0, 25810
section iast_HARDCODED_SECRET_DISABLED
BytebuddyAgent [baseline] (832.381 ms) : 0, 832381
BytebuddyAgent [candidate] (833.784 ms) : 0, 833784
GlobalTracer [baseline] (230.257 ms) : 0, 230257
GlobalTracer [candidate] (230.998 ms) : 0, 230998
AppSec [baseline] (57.718 ms) : 0, 57718
AppSec [candidate] (53.074 ms) : 0, 53074
Remote Config [baseline] (638.519 µs) : 0, 639
Remote Config [candidate] (617.738 µs) : 0, 618
Telemetry [baseline] (8.589 ms) : 0, 8589
Telemetry [candidate] (8.745 ms) : 0, 8745
IAST [baseline] (21.267 ms) : 0, 21267
IAST [candidate] (25.846 ms) : 0, 25846
section iast_TELEMETRY_OFF
BytebuddyAgent [baseline] (829.393 ms) : 0, 829393
BytebuddyAgent [candidate] (831.658 ms) : 0, 831658
GlobalTracer [baseline] (229.802 ms) : 0, 229802
GlobalTracer [candidate] (230.706 ms) : 0, 230706
AppSec [baseline] (58.229 ms) : 0, 58229
AppSec [candidate] (53.897 ms) : 0, 53897
Remote Config [baseline] (653.513 µs) : 0, 654
Remote Config [candidate] (615.741 µs) : 0, 616
Telemetry [baseline] (8.628 ms) : 0, 8628
Telemetry [candidate] (8.47 ms) : 0, 8470
IAST [baseline] (20.982 ms) : 0, 20982
IAST [candidate] (24.833 ms) : 0, 24833
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.46.0-SNAPSHOT~818116f820, baseline=1.46.0-SNAPSHOT~e3793055b5
dateFormat X
axisFormat %s
section baseline
no_agent (1.357 ms) : 1337, 1378
. : milestone, 1357,
appsec (1.742 ms) : 1718, 1765
. : milestone, 1742,
appsec_no_iast (1.768 ms) : 1744, 1793
. : milestone, 1768,
iast (1.509 ms) : 1485, 1533
. : milestone, 1509,
profiling (1.5 ms) : 1476, 1523
. : milestone, 1500,
tracing (1.498 ms) : 1474, 1521
. : milestone, 1498,
section candidate
no_agent (1.37 ms) : 1349, 1390
. : milestone, 1370,
appsec (1.749 ms) : 1726, 1773
. : milestone, 1749,
appsec_no_iast (1.771 ms) : 1745, 1797
. : milestone, 1771,
iast (1.486 ms) : 1462, 1510
. : milestone, 1486,
profiling (1.508 ms) : 1484, 1532
. : milestone, 1508,
tracing (1.487 ms) : 1462, 1512
. : milestone, 1487,
Request duration reports for insecure-bankgantt
title insecure-bank - request duration [CI 0.99] : candidate=1.46.0-SNAPSHOT~818116f820, baseline=1.46.0-SNAPSHOT~e3793055b5
dateFormat X
axisFormat %s
section baseline
no_agent (381.981 µs) : 362, 402
. : milestone, 382,
iast (515.741 µs) : 494, 538
. : milestone, 516,
iast_FULL (744.005 µs) : 721, 767
. : milestone, 744,
iast_GLOBAL (564.757 µs) : 541, 588
. : milestone, 565,
iast_HARDCODED_SECRET_DISABLED (507.351 µs) : 485, 530
. : milestone, 507,
iast_INACTIVE (462.614 µs) : 441, 484
. : milestone, 463,
iast_TELEMETRY_OFF (504.043 µs) : 481, 527
. : milestone, 504,
tracing (450.7 µs) : 430, 471
. : milestone, 451,
section candidate
no_agent (379.938 µs) : 360, 400
. : milestone, 380,
iast (507.792 µs) : 485, 531
. : milestone, 508,
iast_FULL (745.436 µs) : 723, 768
. : milestone, 745,
iast_GLOBAL (568.602 µs) : 544, 594
. : milestone, 569,
iast_HARDCODED_SECRET_DISABLED (517.827 µs) : 494, 541
. : milestone, 518,
iast_INACTIVE (459.47 µs) : 438, 481
. : milestone, 459,
iast_TELEMETRY_OFF (502.4 µs) : 480, 525
. : milestone, 502,
tracing (459.082 µs) : 438, 480
. : milestone, 459,
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.46.0-SNAPSHOT~818116f820, baseline=1.46.0-SNAPSHOT~e3793055b5
dateFormat X
axisFormat %s
section baseline
no_agent (15.279 s) : 15279000, 15279000
. : milestone, 15279000,
appsec (15.065 s) : 15065000, 15065000
. : milestone, 15065000,
iast (18.867 s) : 18867000, 18867000
. : milestone, 18867000,
iast_GLOBAL (18.23 s) : 18230000, 18230000
. : milestone, 18230000,
profiling (15.096 s) : 15096000, 15096000
. : milestone, 15096000,
tracing (15.088 s) : 15088000, 15088000
. : milestone, 15088000,
section candidate
no_agent (15.434 s) : 15434000, 15434000
. : milestone, 15434000,
appsec (15.225 s) : 15225000, 15225000
. : milestone, 15225000,
iast (18.411 s) : 18411000, 18411000
. : milestone, 18411000,
iast_GLOBAL (18.204 s) : 18204000, 18204000
. : milestone, 18204000,
profiling (15.499 s) : 15499000, 15499000
. : milestone, 15499000,
tracing (14.89 s) : 14890000, 14890000
. : milestone, 14890000,
Execution time for tomcatgantt
title tomcat - execution time [CI 0.99] : candidate=1.46.0-SNAPSHOT~818116f820, baseline=1.46.0-SNAPSHOT~e3793055b5
dateFormat X
axisFormat %s
section baseline
no_agent (1.477 ms) : 1466, 1489
. : milestone, 1477,
appsec (2.368 ms) : 2325, 2411
. : milestone, 2368,
iast (2.113 ms) : 2059, 2168
. : milestone, 2113,
iast_GLOBAL (2.157 ms) : 2102, 2213
. : milestone, 2157,
profiling (2.464 ms) : 2273, 2655
. : milestone, 2464,
tracing (1.961 ms) : 1918, 2004
. : milestone, 1961,
section candidate
no_agent (1.471 ms) : 1459, 1482
. : milestone, 1471,
appsec (2.375 ms) : 2331, 2418
. : milestone, 2375,
iast (2.107 ms) : 2052, 2162
. : milestone, 2107,
iast_GLOBAL (2.174 ms) : 2119, 2229
. : milestone, 2174,
profiling (2.006 ms) : 1961, 2051
. : milestone, 2006,
tracing (1.965 ms) : 1923, 2008
. : milestone, 1965,
|
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.
Looks good!
Motivation
When OkHttp might trigger loading of
java.util.logging
we need to delay starting components that make remote calls. However there's no reason to delay starting components which don't actively make remote calls (if they record data that is sent out by another component then that's fine because the collected data will be sent when that component is started after premain.)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: APMS-14233