-
Notifications
You must be signed in to change notification settings - Fork 301
setup.sh should check for JAVA_HOME if version is not 1.8 #8702
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
Signed-off-by: Maciej Obuchowski <[email protected]>
Hi! 👋 Thanks for your pull request! 🎉 To help us review it, please make sure to:
If you need help, please check our contributing guidelines. |
BenchmarksStartupParameters
See matching parameters
SummaryFound 8 performance improvements and 22 performance regressions! Performance is the same for 34 metrics, 7 unstable metrics.
Startup time reports for petclinicgantt
title petclinic - global startup overhead: candidate=1.49.0-SNAPSHOT~1ce7e72239, baseline=1.49.0-SNAPSHOT~069883952e
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.011 s) : 0, 1011142
Total [baseline] (10.54 s) : 0, 10539866
Agent [candidate] (1.056 s) : 0, 1056413
Total [candidate] (10.493 s) : 0, 10493034
section appsec
Agent [baseline] (1.157 s) : 0, 1156819
Total [baseline] (10.759 s) : 0, 10758974
Agent [candidate] (1.205 s) : 0, 1204738
Total [candidate] (10.819 s) : 0, 10819166
section iast
Agent [baseline] (1.148 s) : 0, 1147550
Total [baseline] (10.852 s) : 0, 10851587
Agent [candidate] (1.188 s) : 0, 1188459
Total [candidate] (11.051 s) : 0, 11050700
section profiling
Agent [baseline] (1.277 s) : 0, 1276502
Total [baseline] (10.946 s) : 0, 10946351
Agent [candidate] (1.299 s) : 0, 1298765
Total [candidate] (10.958 s) : 0, 10958053
gantt
title petclinic - break down per module: candidate=1.49.0-SNAPSHOT~1ce7e72239, baseline=1.49.0-SNAPSHOT~069883952e
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (673.085 ms) : 0, 673085
BytebuddyAgent [candidate] (721.32 ms) : 0, 721320
GlobalTracer [baseline] (241.229 ms) : 0, 241229
GlobalTracer [candidate] (239.45 ms) : 0, 239450
AppSec [baseline] (55.104 ms) : 0, 55104
AppSec [candidate] (55.507 ms) : 0, 55507
Debugger [baseline] (6.174 ms) : 0, 6174
Debugger [candidate] (4.424 ms) : 0, 4424
Remote Config [baseline] (702.857 µs) : 0, 703
Remote Config [candidate] (714.653 µs) : 0, 715
Telemetry [baseline] (11.429 ms) : 0, 11429
Telemetry [candidate] (11.541 ms) : 0, 11541
section appsec
BytebuddyAgent [baseline] (693.793 ms) : 0, 693793
BytebuddyAgent [candidate] (743.877 ms) : 0, 743877
GlobalTracer [baseline] (238.004 ms) : 0, 238004
GlobalTracer [candidate] (237.537 ms) : 0, 237537
IAST [baseline] (21.836 ms) : 0, 21836
IAST [candidate] (21.999 ms) : 0, 21999
AppSec [baseline] (175.838 ms) : 0, 175838
AppSec [candidate] (175.37 ms) : 0, 175370
Debugger [baseline] (5.881 ms) : 0, 5881
Debugger [candidate] (4.596 ms) : 0, 4596
Remote Config [baseline] (641.009 µs) : 0, 641
Remote Config [candidate] (634.401 µs) : 0, 634
Telemetry [baseline] (8.173 ms) : 0, 8173
Telemetry [candidate] (8.18 ms) : 0, 8180
section iast
BytebuddyAgent [baseline] (797.493 ms) : 0, 797493
BytebuddyAgent [candidate] (841.288 ms) : 0, 841288
GlobalTracer [baseline] (231.977 ms) : 0, 231977
GlobalTracer [candidate] (230.543 ms) : 0, 230543
IAST [baseline] (23.033 ms) : 0, 23033
IAST [candidate] (23.03 ms) : 0, 23030
AppSec [baseline] (56.984 ms) : 0, 56984
AppSec [candidate] (56.493 ms) : 0, 56493
Debugger [baseline] (5.904 ms) : 0, 5904
Debugger [candidate] (4.266 ms) : 0, 4266
Remote Config [baseline] (595.806 µs) : 0, 596
Remote Config [candidate] (616.812 µs) : 0, 617
Telemetry [baseline] (7.97 ms) : 0, 7970
Telemetry [candidate] (8.742 ms) : 0, 8742
section profiling
BytebuddyAgent [baseline] (677.344 ms) : 0, 677344
BytebuddyAgent [candidate] (716.279 ms) : 0, 716279
GlobalTracer [baseline] (380.638 ms) : 0, 380638
GlobalTracer [candidate] (362.164 ms) : 0, 362164
AppSec [baseline] (54.671 ms) : 0, 54671
AppSec [candidate] (53.401 ms) : 0, 53401
Debugger [baseline] (6.138 ms) : 0, 6138
Debugger [candidate] (4.392 ms) : 0, 4392
Remote Config [baseline] (665.823 µs) : 0, 666
Remote Config [candidate] (664.778 µs) : 0, 665
Telemetry [baseline] (8.295 ms) : 0, 8295
Telemetry [candidate] (8.996 ms) : 0, 8996
ProfilingAgent [baseline] (97.869 ms) : 0, 97869
ProfilingAgent [candidate] (102.287 ms) : 0, 102287
Profiling [baseline] (97.894 ms) : 0, 97894
Profiling [candidate] (102.312 ms) : 0, 102312
Startup time reports for insecure-bankgantt
title insecure-bank - global startup overhead: candidate=1.49.0-SNAPSHOT~1ce7e72239, baseline=1.49.0-SNAPSHOT~069883952e
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.008 s) : 0, 1007829
Total [baseline] (8.636 s) : 0, 8636457
Agent [candidate] (1.057 s) : 0, 1057103
Total [candidate] (8.691 s) : 0, 8691498
section iast
Agent [baseline] (1.138 s) : 0, 1138396
Total [baseline] (9.252 s) : 0, 9251940
Agent [candidate] (1.196 s) : 0, 1195854
Total [candidate] (9.267 s) : 0, 9266858
section iast_HARDCODED_SECRET_DISABLED
Agent [baseline] (1.147 s) : 0, 1147158
Total [baseline] (9.222 s) : 0, 9222274
Agent [candidate] (1.187 s) : 0, 1186999
Total [candidate] (9.243 s) : 0, 9242592
section iast_TELEMETRY_OFF
Agent [baseline] (1.141 s) : 0, 1140543
Total [baseline] (9.189 s) : 0, 9189382
Agent [candidate] (1.194 s) : 0, 1194042
Total [candidate] (9.29 s) : 0, 9289794
gantt
title insecure-bank - break down per module: candidate=1.49.0-SNAPSHOT~1ce7e72239, baseline=1.49.0-SNAPSHOT~069883952e
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (672.285 ms) : 0, 672285
BytebuddyAgent [candidate] (721.271 ms) : 0, 721271
GlobalTracer [baseline] (241.079 ms) : 0, 241079
GlobalTracer [candidate] (239.904 ms) : 0, 239904
AppSec [baseline] (54.784 ms) : 0, 54784
AppSec [candidate] (55.263 ms) : 0, 55263
Debugger [baseline] (6.159 ms) : 0, 6159
Debugger [candidate] (4.382 ms) : 0, 4382
Remote Config [baseline] (715.23 µs) : 0, 715
Remote Config [candidate] (2.132 ms) : 0, 2132
Telemetry [baseline] (9.239 ms) : 0, 9239
Telemetry [candidate] (10.662 ms) : 0, 10662
section iast
BytebuddyAgent [baseline] (791.178 ms) : 0, 791178
BytebuddyAgent [candidate] (847.403 ms) : 0, 847403
GlobalTracer [baseline] (230.635 ms) : 0, 230635
GlobalTracer [candidate] (231.748 ms) : 0, 231748
IAST [baseline] (22.756 ms) : 0, 22756
IAST [candidate] (23.891 ms) : 0, 23891
AppSec [baseline] (56.166 ms) : 0, 56166
AppSec [candidate] (55.513 ms) : 0, 55513
Debugger [baseline] (5.854 ms) : 0, 5854
Debugger [candidate] (4.319 ms) : 0, 4319
Remote Config [baseline] (584.83 µs) : 0, 585
Remote Config [candidate] (611.541 µs) : 0, 612
Telemetry [baseline] (7.844 ms) : 0, 7844
Telemetry [candidate] (8.819 ms) : 0, 8819
section iast_HARDCODED_SECRET_DISABLED
BytebuddyAgent [baseline] (797.13 ms) : 0, 797130
BytebuddyAgent [candidate] (840.965 ms) : 0, 840965
GlobalTracer [baseline] (231.721 ms) : 0, 231721
GlobalTracer [candidate] (230.155 ms) : 0, 230155
IAST [baseline] (23.059 ms) : 0, 23059
IAST [candidate] (23.647 ms) : 0, 23647
AppSec [baseline] (56.966 ms) : 0, 56966
AppSec [candidate] (55.273 ms) : 0, 55273
Debugger [baseline] (6.028 ms) : 0, 6028
Debugger [candidate] (4.206 ms) : 0, 4206
Remote Config [baseline] (597.689 µs) : 0, 598
Remote Config [candidate] (607.342 µs) : 0, 607
Telemetry [baseline] (8.085 ms) : 0, 8085
Telemetry [candidate] (8.739 ms) : 0, 8739
section iast_TELEMETRY_OFF
BytebuddyAgent [baseline] (792.576 ms) : 0, 792576
BytebuddyAgent [candidate] (846.353 ms) : 0, 846353
GlobalTracer [baseline] (231.006 ms) : 0, 231006
GlobalTracer [candidate] (231.892 ms) : 0, 231892
IAST [baseline] (22.337 ms) : 0, 22337
IAST [candidate] (22.722 ms) : 0, 22722
AppSec [baseline] (56.843 ms) : 0, 56843
AppSec [candidate] (55.986 ms) : 0, 55986
Debugger [baseline] (5.928 ms) : 0, 5928
Debugger [candidate] (4.169 ms) : 0, 4169
Remote Config [baseline] (599.045 µs) : 0, 599
Remote Config [candidate] (627.971 µs) : 0, 628
Telemetry [baseline] (7.79 ms) : 0, 7790
Telemetry [candidate] (8.684 ms) : 0, 8684
LoadParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 12 metrics, 18 unstable metrics. Request duration reports for insecure-bankgantt
title insecure-bank - request duration [CI 0.99] : candidate=1.49.0-SNAPSHOT~1ce7e72239, baseline=1.49.0-SNAPSHOT~069883952e
dateFormat X
axisFormat %s
section baseline
no_agent (378.505 µs) : 358, 399
. : milestone, 379,
iast (513.481 µs) : 490, 537
. : milestone, 513,
iast_FULL (728.993 µs) : 706, 752
. : milestone, 729,
iast_GLOBAL (557.034 µs) : 534, 580
. : milestone, 557,
iast_HARDCODED_SECRET_DISABLED (510.846 µs) : 488, 534
. : milestone, 511,
iast_INACTIVE (466.543 µs) : 444, 489
. : milestone, 467,
iast_TELEMETRY_OFF (508.273 µs) : 485, 531
. : milestone, 508,
tracing (458.16 µs) : 436, 480
. : milestone, 458,
section candidate
no_agent (380.055 µs) : 360, 400
. : milestone, 380,
iast (511.49 µs) : 489, 534
. : milestone, 511,
iast_FULL (727.549 µs) : 705, 750
. : milestone, 728,
iast_GLOBAL (558.454 µs) : 536, 581
. : milestone, 558,
iast_HARDCODED_SECRET_DISABLED (511.254 µs) : 490, 533
. : milestone, 511,
iast_INACTIVE (464.368 µs) : 442, 486
. : milestone, 464,
iast_TELEMETRY_OFF (499.416 µs) : 477, 521
. : milestone, 499,
tracing (459.722 µs) : 439, 481
. : milestone, 460,
Request duration reports for petclinicgantt
title petclinic - request duration [CI 0.99] : candidate=1.49.0-SNAPSHOT~1ce7e72239, baseline=1.49.0-SNAPSHOT~069883952e
dateFormat X
axisFormat %s
section baseline
no_agent (1.346 ms) : 1326, 1367
. : milestone, 1346,
appsec (1.713 ms) : 1690, 1736
. : milestone, 1713,
appsec_no_iast (1.712 ms) : 1689, 1736
. : milestone, 1712,
code_origins (1.663 ms) : 1635, 1690
. : milestone, 1663,
iast (1.505 ms) : 1479, 1530
. : milestone, 1505,
profiling (1.557 ms) : 1532, 1583
. : milestone, 1557,
tracing (1.471 ms) : 1445, 1496
. : milestone, 1471,
section candidate
no_agent (1.357 ms) : 1337, 1377
. : milestone, 1357,
appsec (1.723 ms) : 1699, 1746
. : milestone, 1723,
appsec_no_iast (1.723 ms) : 1699, 1746
. : milestone, 1723,
code_origins (1.669 ms) : 1642, 1696
. : milestone, 1669,
iast (1.499 ms) : 1475, 1524
. : milestone, 1499,
profiling (1.508 ms) : 1484, 1532
. : milestone, 1508,
tracing (1.503 ms) : 1478, 1528
. : milestone, 1503,
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.49.0-SNAPSHOT~1ce7e72239, baseline=1.49.0-SNAPSHOT~069883952e
dateFormat X
axisFormat %s
section baseline
no_agent (14.835 s) : 14835000, 14835000
. : milestone, 14835000,
appsec (15.038 s) : 15038000, 15038000
. : milestone, 15038000,
iast (18.617 s) : 18617000, 18617000
. : milestone, 18617000,
iast_GLOBAL (18.125 s) : 18125000, 18125000
. : milestone, 18125000,
profiling (15.124 s) : 15124000, 15124000
. : milestone, 15124000,
tracing (15.001 s) : 15001000, 15001000
. : milestone, 15001000,
section candidate
no_agent (15.66 s) : 15660000, 15660000
. : milestone, 15660000,
appsec (15.024 s) : 15024000, 15024000
. : milestone, 15024000,
iast (18.632 s) : 18632000, 18632000
. : milestone, 18632000,
iast_GLOBAL (17.632 s) : 17632000, 17632000
. : milestone, 17632000,
profiling (14.994 s) : 14994000, 14994000
. : milestone, 14994000,
tracing (15.004 s) : 15004000, 15004000
. : milestone, 15004000,
Execution time for tomcatgantt
title tomcat - execution time [CI 0.99] : candidate=1.49.0-SNAPSHOT~1ce7e72239, baseline=1.49.0-SNAPSHOT~069883952e
dateFormat X
axisFormat %s
section baseline
no_agent (1.471 ms) : 1459, 1482
. : milestone, 1471,
appsec (2.372 ms) : 2325, 2419
. : milestone, 2372,
iast (2.146 ms) : 2087, 2205
. : milestone, 2146,
iast_GLOBAL (2.193 ms) : 2134, 2252
. : milestone, 2193,
profiling (1.993 ms) : 1947, 2040
. : milestone, 1993,
tracing (1.973 ms) : 1928, 2018
. : milestone, 1973,
section candidate
no_agent (1.468 ms) : 1456, 1479
. : milestone, 1468,
appsec (2.371 ms) : 2324, 2418
. : milestone, 2371,
iast (2.158 ms) : 2099, 2217
. : milestone, 2158,
iast_GLOBAL (2.203 ms) : 2143, 2263
. : milestone, 2203,
profiling (1.995 ms) : 1948, 2042
. : milestone, 1995,
tracing (1.975 ms) : 1929, 2020
. : milestone, 1975,
|
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.
Thanks the improvement! 👍
In case where java
is in the path and not Java 8, is the error message still relevant? 🤔 (ie asking to set the JAVA_HOME
env var?)
And what about the Windows version of this script, does it need to be updated too?
@PerfectSlayer yes, with
Unfortunately I have no way of testing the windows script - should I just update it in case? |
I pushed an update for the script. I don’t maintain the Windows one either but maybe @lucaspimentel can? |
What Does This Do
The setup.sh check needs an additional condition if
java
binary is not Java 8. Else, gradle fails: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]