-
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
JVM version output can omit minor or patch suffixes #7068
Conversation
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 for the fix! ❤️
@@ -10,7 +10,7 @@ function check-jvm() { | |||
if [ -z ${!JAVA_HOME_NAME} ]; then | |||
echo "❌ $JAVA_HOME_NAME is not set. Please set $JAVA_HOME_NAME to refer to a JDK $EXPECTED_JAVA_VERSION installation." >&2 | |||
exit 1 | |||
elif ! ${!JAVA_HOME_NAME}/bin/java -version 2>&1 | grep -q "version \"$EXPECTED_JAVA_VERSION\." ; then | |||
elif ! ${!JAVA_HOME_NAME}/bin/java -version 2>&1 | grep -q "version \"$EXPECTED_JAVA_VERSION" ; then |
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.
I guess Java 80
is far enough in the future for this to be a problem 😉
BenchmarksStartupParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 47 metrics, 16 unstable metrics. Startup time reports for petclinicgantt
title petclinic - global startup overhead: candidate=1.39.0-SNAPSHOT~61e8eefc93, baseline=1.39.0-SNAPSHOT~39a68b33b5
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.054 s) : 0, 1053682
Total [baseline] (10.312 s) : 0, 10312302
Agent [candidate] (1.055 s) : 0, 1054576
Total [candidate] (10.352 s) : 0, 10351759
section appsec
Agent [baseline] (1.172 s) : 0, 1171514
Total [baseline] (10.565 s) : 0, 10565264
Agent [candidate] (1.174 s) : 0, 1173588
Total [candidate] (10.524 s) : 0, 10523623
section iast
Agent [baseline] (1.185 s) : 0, 1185445
Total [baseline] (10.784 s) : 0, 10783801
Agent [candidate] (1.192 s) : 0, 1191693
Total [candidate] (10.874 s) : 0, 10873919
section profiling
Agent [baseline] (1.245 s) : 0, 1244899
Total [baseline] (10.626 s) : 0, 10625673
Agent [candidate] (1.247 s) : 0, 1247002
Total [candidate] (10.603 s) : 0, 10602551
gantt
title petclinic - break down per module: candidate=1.39.0-SNAPSHOT~61e8eefc93, baseline=1.39.0-SNAPSHOT~39a68b33b5
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (673.521 ms) : 0, 673521
BytebuddyAgent [candidate] (673.17 ms) : 0, 673170
GlobalTracer [baseline] (308.022 ms) : 0, 308022
GlobalTracer [candidate] (309.123 ms) : 0, 309123
AppSec [baseline] (50.496 ms) : 0, 50496
AppSec [candidate] (50.645 ms) : 0, 50645
Remote Config [baseline] (701.3 µs) : 0, 701
Remote Config [candidate] (701.912 µs) : 0, 702
Telemetry [baseline] (7.399 ms) : 0, 7399
Telemetry [candidate] (7.393 ms) : 0, 7393
section appsec
BytebuddyAgent [baseline] (680.897 ms) : 0, 680897
BytebuddyAgent [candidate] (683.194 ms) : 0, 683194
GlobalTracer [baseline] (300.945 ms) : 0, 300945
GlobalTracer [candidate] (300.844 ms) : 0, 300844
AppSec [baseline] (155.343 ms) : 0, 155343
AppSec [candidate] (155.7 ms) : 0, 155700
Remote Config [baseline] (609.486 µs) : 0, 609
Remote Config [candidate] (610.463 µs) : 0, 610
Telemetry [baseline] (9.147 ms) : 0, 9147
Telemetry [candidate] (8.227 ms) : 0, 8227
IAST [baseline] (22.048 ms) : 0, 22048
IAST [candidate] (22.834 ms) : 0, 22834
section iast
BytebuddyAgent [baseline] (789.456 ms) : 0, 789456
BytebuddyAgent [candidate] (794.445 ms) : 0, 794445
GlobalTracer [baseline] (298.394 ms) : 0, 298394
GlobalTracer [candidate] (300.308 ms) : 0, 300308
AppSec [baseline] (50.467 ms) : 0, 50467
AppSec [candidate] (51.3 ms) : 0, 51300
Remote Config [baseline] (584.813 µs) : 0, 585
Remote Config [candidate] (620.252 µs) : 0, 620
Telemetry [baseline] (9.536 ms) : 0, 9536
Telemetry [candidate] (7.015 ms) : 0, 7015
IAST [baseline] (23.415 ms) : 0, 23415
IAST [candidate] (24.324 ms) : 0, 24324
section profiling
BytebuddyAgent [baseline] (662.852 ms) : 0, 662852
BytebuddyAgent [candidate] (664.373 ms) : 0, 664373
GlobalTracer [baseline] (390.482 ms) : 0, 390482
GlobalTracer [candidate] (391.12 ms) : 0, 391120
AppSec [baseline] (51.221 ms) : 0, 51221
AppSec [candidate] (51.23 ms) : 0, 51230
Remote Config [baseline] (680.105 µs) : 0, 680
Remote Config [candidate] (699.099 µs) : 0, 699
Telemetry [baseline] (7.21 ms) : 0, 7210
Telemetry [candidate] (7.223 ms) : 0, 7223
ProfilingAgent [baseline] (95.262 ms) : 0, 95262
ProfilingAgent [candidate] (95.1 ms) : 0, 95100
Profiling [baseline] (95.286 ms) : 0, 95286
Profiling [candidate] (95.124 ms) : 0, 95124
Startup time reports for insecure-bankgantt
title insecure-bank - global startup overhead: candidate=1.39.0-SNAPSHOT~61e8eefc93, baseline=1.39.0-SNAPSHOT~39a68b33b5
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.054 s) : 0, 1054285
Total [baseline] (8.5 s) : 0, 8500485
Agent [candidate] (1.051 s) : 0, 1050515
Total [candidate] (8.529 s) : 0, 8529166
section iast
Agent [baseline] (1.178 s) : 0, 1177708
Total [baseline] (9.039 s) : 0, 9039489
Agent [candidate] (1.192 s) : 0, 1191568
Total [candidate] (9.031 s) : 0, 9030742
section iast_HARDCODED_SECRET_DISABLED
Agent [baseline] (1.186 s) : 0, 1185846
Total [baseline] (8.974 s) : 0, 8974161
Agent [candidate] (1.175 s) : 0, 1175313
Total [candidate] (8.969 s) : 0, 8968981
section iast_TELEMETRY_OFF
Agent [baseline] (1.173 s) : 0, 1173205
Total [baseline] (8.996 s) : 0, 8996379
Agent [candidate] (1.173 s) : 0, 1172716
Total [candidate] (9.016 s) : 0, 9015643
gantt
title insecure-bank - break down per module: candidate=1.39.0-SNAPSHOT~61e8eefc93, baseline=1.39.0-SNAPSHOT~39a68b33b5
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (673.153 ms) : 0, 673153
BytebuddyAgent [candidate] (668.953 ms) : 0, 668953
GlobalTracer [baseline] (308.689 ms) : 0, 308689
GlobalTracer [candidate] (309.377 ms) : 0, 309377
AppSec [baseline] (50.761 ms) : 0, 50761
AppSec [candidate] (50.587 ms) : 0, 50587
Remote Config [baseline] (697.151 µs) : 0, 697
Remote Config [candidate] (695.257 µs) : 0, 695
Telemetry [baseline] (7.439 ms) : 0, 7439
Telemetry [candidate] (7.443 ms) : 0, 7443
section iast
BytebuddyAgent [baseline] (783.797 ms) : 0, 783797
BytebuddyAgent [candidate] (794.825 ms) : 0, 794825
GlobalTracer [baseline] (297.022 ms) : 0, 297022
GlobalTracer [candidate] (300.332 ms) : 0, 300332
AppSec [baseline] (51.733 ms) : 0, 51733
AppSec [candidate] (48.978 ms) : 0, 48978
Remote Config [baseline] (583.771 µs) : 0, 584
Remote Config [candidate] (582.253 µs) : 0, 582
Telemetry [baseline] (7.805 ms) : 0, 7805
Telemetry [candidate] (7.826 ms) : 0, 7826
IAST [baseline] (23.303 ms) : 0, 23303
IAST [candidate] (25.331 ms) : 0, 25331
section iast_HARDCODED_SECRET_DISABLED
BytebuddyAgent [baseline] (789.175 ms) : 0, 789175
BytebuddyAgent [candidate] (782.501 ms) : 0, 782501
GlobalTracer [baseline] (299.108 ms) : 0, 299108
GlobalTracer [candidate] (296.78 ms) : 0, 296780
AppSec [baseline] (50.495 ms) : 0, 50495
AppSec [candidate] (49.532 ms) : 0, 49532
Remote Config [baseline] (604.058 µs) : 0, 604
Remote Config [candidate] (590.553 µs) : 0, 591
Telemetry [baseline] (7.836 ms) : 0, 7836
Telemetry [candidate] (9.582 ms) : 0, 9582
IAST [baseline] (25.002 ms) : 0, 25002
IAST [candidate] (22.739 ms) : 0, 22739
section iast_TELEMETRY_OFF
BytebuddyAgent [baseline] (780.546 ms) : 0, 780546
BytebuddyAgent [candidate] (780.579 ms) : 0, 780579
GlobalTracer [baseline] (297.681 ms) : 0, 297681
GlobalTracer [candidate] (296.813 ms) : 0, 296813
AppSec [baseline] (46.996 ms) : 0, 46996
AppSec [candidate] (48.377 ms) : 0, 48377
Remote Config [baseline] (611.448 µs) : 0, 611
Remote Config [candidate] (598.51 µs) : 0, 599
Telemetry [baseline] (7.686 ms) : 0, 7686
Telemetry [candidate] (6.89 ms) : 0, 6890
IAST [baseline] (26.182 ms) : 0, 26182
IAST [candidate] (25.924 ms) : 0, 25924
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 insecure-bankgantt
title insecure-bank - request duration [CI 0.99] : candidate=1.39.0-SNAPSHOT~61e8eefc93, baseline=1.39.0-SNAPSHOT~39a68b33b5
dateFormat X
axisFormat %s
section baseline
no_agent (362.912 µs) : 343, 383
. : milestone, 363,
iast (481.903 µs) : 461, 503
. : milestone, 482,
iast_FULL (549.022 µs) : 528, 570
. : milestone, 549,
iast_GLOBAL (502.84 µs) : 482, 524
. : milestone, 503,
iast_HARDCODED_SECRET_DISABLED (486.066 µs) : 465, 507
. : milestone, 486,
iast_INACTIVE (456.795 µs) : 435, 479
. : milestone, 457,
iast_TELEMETRY_OFF (478.008 µs) : 456, 500
. : milestone, 478,
tracing (445.961 µs) : 426, 466
. : milestone, 446,
section candidate
no_agent (368.725 µs) : 349, 388
. : milestone, 369,
iast (483.158 µs) : 462, 504
. : milestone, 483,
iast_FULL (550.91 µs) : 530, 572
. : milestone, 551,
iast_GLOBAL (503.687 µs) : 481, 526
. : milestone, 504,
iast_HARDCODED_SECRET_DISABLED (480.882 µs) : 460, 502
. : milestone, 481,
iast_INACTIVE (463.374 µs) : 441, 486
. : milestone, 463,
iast_TELEMETRY_OFF (477.227 µs) : 456, 499
. : milestone, 477,
tracing (441.525 µs) : 421, 462
. : milestone, 442,
Request duration reports for petclinicgantt
title petclinic - request duration [CI 0.99] : candidate=1.39.0-SNAPSHOT~61e8eefc93, baseline=1.39.0-SNAPSHOT~39a68b33b5
dateFormat X
axisFormat %s
section baseline
no_agent (1.339 ms) : 1320, 1359
. : milestone, 1339,
appsec (1.732 ms) : 1709, 1755
. : milestone, 1732,
appsec_no_iast (1.717 ms) : 1693, 1741
. : milestone, 1717,
iast (1.51 ms) : 1488, 1532
. : milestone, 1510,
profiling (1.478 ms) : 1454, 1503
. : milestone, 1478,
tracing (1.475 ms) : 1450, 1500
. : milestone, 1475,
section candidate
no_agent (1.351 ms) : 1331, 1371
. : milestone, 1351,
appsec (1.695 ms) : 1672, 1718
. : milestone, 1695,
appsec_no_iast (1.737 ms) : 1714, 1761
. : milestone, 1737,
iast (1.482 ms) : 1458, 1505
. : milestone, 1482,
profiling (1.518 ms) : 1492, 1544
. : milestone, 1518,
tracing (1.459 ms) : 1434, 1484
. : milestone, 1459,
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.39.0-SNAPSHOT~61e8eefc93, baseline=1.39.0-SNAPSHOT~39a68b33b5
dateFormat X
axisFormat %s
section baseline
no_agent (14.962 s) : 14962000, 14962000
. : milestone, 14962000,
appsec (14.967 s) : 14967000, 14967000
. : milestone, 14967000,
iast (18.566 s) : 18566000, 18566000
. : milestone, 18566000,
iast_GLOBAL (17.995 s) : 17995000, 17995000
. : milestone, 17995000,
profiling (15.527 s) : 15527000, 15527000
. : milestone, 15527000,
tracing (15.02 s) : 15020000, 15020000
. : milestone, 15020000,
section candidate
no_agent (15.209 s) : 15209000, 15209000
. : milestone, 15209000,
appsec (15.247 s) : 15247000, 15247000
. : milestone, 15247000,
iast (18.795 s) : 18795000, 18795000
. : milestone, 18795000,
iast_GLOBAL (18.061 s) : 18061000, 18061000
. : milestone, 18061000,
profiling (15.907 s) : 15907000, 15907000
. : milestone, 15907000,
tracing (15.193 s) : 15193000, 15193000
. : milestone, 15193000,
Execution time for tomcatgantt
title tomcat - execution time [CI 0.99] : candidate=1.39.0-SNAPSHOT~61e8eefc93, baseline=1.39.0-SNAPSHOT~39a68b33b5
dateFormat X
axisFormat %s
section baseline
no_agent (1.466 ms) : 1454, 1477
. : milestone, 1466,
appsec (2.228 ms) : 2193, 2263
. : milestone, 2228,
iast (1.992 ms) : 1949, 2034
. : milestone, 1992,
iast_GLOBAL (2.018 ms) : 1975, 2060
. : milestone, 2018,
profiling (1.864 ms) : 1831, 1898
. : milestone, 1864,
tracing (1.84 ms) : 1807, 1873
. : milestone, 1840,
section candidate
no_agent (1.461 ms) : 1449, 1472
. : milestone, 1461,
appsec (2.221 ms) : 2186, 2256
. : milestone, 2221,
iast (1.97 ms) : 1929, 2012
. : milestone, 1970,
iast_GLOBAL (2.014 ms) : 1972, 2056
. : milestone, 2014,
profiling (1.863 ms) : 1828, 1897
. : milestone, 1863,
tracing (1.841 ms) : 1808, 1873
. : milestone, 1841,
|
What Does This Do
Patches the setup script to account for release numbers with minor / patch omitted.
Motivation
The
./setup.sh
script presumes that the version number output ofjava -version
will follow semver conventions, so major versions will be followed by a.
. In the case of some JDKs, i.e. Temurin 17, the output only includes the major number:Additional Notes
Jira ticket: will backfill if needed