-
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
Replace JAVA_TOOL_OPTIONS by jvm.options OpenLiberty server configuration file #8031
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
PerfectSlayer
added
comp: testing
Testing
tag: no release notes
Changes to exclude from release notes
inst: opentelemetry
OpenTelemetry instrumentation
labels
Nov 28, 2024
PerfectSlayer
changed the title
Replace JAVA_TOOL_OPTIONS by jvm.options openliberty server configuration file
Replace JAVA_TOOL_OPTIONS by jvm.options OpenLiberty server configuration file
Nov 28, 2024
BenchmarksStartupParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 55 metrics, 8 unstable metrics. Startup time reports for insecure-bankgantt
title insecure-bank - global startup overhead: candidate=1.44.0-SNAPSHOT~e73523127f, baseline=1.44.0-SNAPSHOT~508c314d67
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.091 s) : 0, 1091035
Total [baseline] (8.656 s) : 0, 8656189
Agent [candidate] (1.092 s) : 0, 1092005
Total [candidate] (8.639 s) : 0, 8639393
section iast
Agent [baseline] (1.217 s) : 0, 1217473
Total [baseline] (9.231 s) : 0, 9230760
Agent [candidate] (1.221 s) : 0, 1221118
Total [candidate] (9.203 s) : 0, 9203379
section iast_HARDCODED_SECRET_DISABLED
Agent [baseline] (1.218 s) : 0, 1217948
Total [baseline] (9.169 s) : 0, 9169023
Agent [candidate] (1.237 s) : 0, 1237103
Total [candidate] (9.3 s) : 0, 9300450
section iast_TELEMETRY_OFF
Agent [baseline] (1.22 s) : 0, 1220318
Total [baseline] (9.167 s) : 0, 9167314
Agent [candidate] (1.216 s) : 0, 1215547
Total [candidate] (9.169 s) : 0, 9168822
gantt
title insecure-bank - break down per module: candidate=1.44.0-SNAPSHOT~e73523127f, baseline=1.44.0-SNAPSHOT~508c314d67
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (694.581 ms) : 0, 694581
BytebuddyAgent [candidate] (694.981 ms) : 0, 694981
GlobalTracer [baseline] (318.853 ms) : 0, 318853
GlobalTracer [candidate] (317.561 ms) : 0, 317561
AppSec [baseline] (54.768 ms) : 0, 54768
AppSec [candidate] (55.042 ms) : 0, 55042
Remote Config [baseline] (690.196 µs) : 0, 690
Remote Config [candidate] (677.376 µs) : 0, 677
Telemetry [baseline] (8.422 ms) : 0, 8422
Telemetry [candidate] (9.877 ms) : 0, 9877
section iast
BytebuddyAgent [baseline] (810.811 ms) : 0, 810811
BytebuddyAgent [candidate] (812.805 ms) : 0, 812805
GlobalTracer [baseline] (305.525 ms) : 0, 305525
GlobalTracer [candidate] (307.0 ms) : 0, 307000
AppSec [baseline] (56.807 ms) : 0, 56807
AppSec [candidate] (58.446 ms) : 0, 58446
IAST [baseline] (22.479 ms) : 0, 22479
IAST [candidate] (20.901 ms) : 0, 20901
Remote Config [baseline] (620.877 µs) : 0, 621
Remote Config [candidate] (629.302 µs) : 0, 629
Telemetry [baseline] (7.5 ms) : 0, 7500
Telemetry [candidate] (7.557 ms) : 0, 7557
section iast_HARDCODED_SECRET_DISABLED
BytebuddyAgent [baseline] (811.513 ms) : 0, 811513
BytebuddyAgent [candidate] (825.437 ms) : 0, 825437
GlobalTracer [baseline] (305.816 ms) : 0, 305816
GlobalTracer [candidate] (309.806 ms) : 0, 309806
AppSec [baseline] (56.834 ms) : 0, 56834
AppSec [candidate] (58.524 ms) : 0, 58524
IAST [baseline] (21.809 ms) : 0, 21809
IAST [candidate] (21.093 ms) : 0, 21093
Remote Config [baseline] (633.186 µs) : 0, 633
Remote Config [candidate] (642.037 µs) : 0, 642
Telemetry [baseline] (7.541 ms) : 0, 7541
Telemetry [candidate] (7.651 ms) : 0, 7651
section iast_TELEMETRY_OFF
BytebuddyAgent [baseline] (814.012 ms) : 0, 814012
BytebuddyAgent [candidate] (808.854 ms) : 0, 808854
GlobalTracer [baseline] (305.616 ms) : 0, 305616
GlobalTracer [candidate] (306.27 ms) : 0, 306270
AppSec [baseline] (58.283 ms) : 0, 58283
AppSec [candidate] (58.106 ms) : 0, 58106
IAST [baseline] (20.498 ms) : 0, 20498
IAST [candidate] (20.486 ms) : 0, 20486
Remote Config [baseline] (632.156 µs) : 0, 632
Remote Config [candidate] (629.707 µs) : 0, 630
Telemetry [baseline] (7.451 ms) : 0, 7451
Telemetry [candidate] (7.455 ms) : 0, 7455
Startup time reports for petclinicgantt
title petclinic - global startup overhead: candidate=1.44.0-SNAPSHOT~e73523127f, baseline=1.44.0-SNAPSHOT~508c314d67
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.09 s) : 0, 1090394
Total [baseline] (10.448 s) : 0, 10448251
Agent [candidate] (1.098 s) : 0, 1098476
Total [candidate] (10.399 s) : 0, 10398871
section appsec
Agent [baseline] (1.223 s) : 0, 1222893
Total [baseline] (10.696 s) : 0, 10696075
Agent [candidate] (1.224 s) : 0, 1224310
Total [candidate] (10.731 s) : 0, 10731353
section iast
Agent [baseline] (1.217 s) : 0, 1217080
Total [baseline] (10.95 s) : 0, 10949843
Agent [candidate] (1.22 s) : 0, 1219795
Total [candidate] (11.029 s) : 0, 11028607
section profiling
Agent [baseline] (1.312 s) : 0, 1312046
Total [baseline] (10.755 s) : 0, 10754772
Agent [candidate] (1.325 s) : 0, 1324560
Total [candidate] (10.798 s) : 0, 10798063
gantt
title petclinic - break down per module: candidate=1.44.0-SNAPSHOT~e73523127f, baseline=1.44.0-SNAPSHOT~508c314d67
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (693.085 ms) : 0, 693085
BytebuddyAgent [candidate] (697.61 ms) : 0, 697610
GlobalTracer [baseline] (317.518 ms) : 0, 317518
GlobalTracer [candidate] (319.564 ms) : 0, 319564
AppSec [baseline] (54.838 ms) : 0, 54838
AppSec [candidate] (54.741 ms) : 0, 54741
Remote Config [baseline] (689.233 µs) : 0, 689
Remote Config [candidate] (679.352 µs) : 0, 679
Telemetry [baseline] (10.563 ms) : 0, 10563
Telemetry [candidate] (12.047 ms) : 0, 12047
section appsec
BytebuddyAgent [baseline] (709.788 ms) : 0, 709788
BytebuddyAgent [candidate] (710.71 ms) : 0, 710710
GlobalTracer [baseline] (313.991 ms) : 0, 313991
GlobalTracer [candidate] (314.317 ms) : 0, 314317
AppSec [baseline] (167.479 ms) : 0, 167479
AppSec [candidate] (166.511 ms) : 0, 166511
Remote Config [baseline] (639.335 µs) : 0, 639
Remote Config [candidate] (634.813 µs) : 0, 635
Telemetry [baseline] (7.416 ms) : 0, 7416
Telemetry [candidate] (8.924 ms) : 0, 8924
IAST [baseline] (19.772 ms) : 0, 19772
IAST [candidate] (18.94 ms) : 0, 18940
section iast
BytebuddyAgent [baseline] (811.035 ms) : 0, 811035
BytebuddyAgent [candidate] (812.098 ms) : 0, 812098
GlobalTracer [baseline] (306.078 ms) : 0, 306078
GlobalTracer [candidate] (306.864 ms) : 0, 306864
AppSec [baseline] (57.267 ms) : 0, 57267
AppSec [candidate] (58.039 ms) : 0, 58039
Remote Config [baseline] (634.87 µs) : 0, 635
Remote Config [candidate] (652.455 µs) : 0, 652
Telemetry [baseline] (7.504 ms) : 0, 7504
Telemetry [candidate] (7.515 ms) : 0, 7515
IAST [baseline] (20.845 ms) : 0, 20845
IAST [candidate] (20.855 ms) : 0, 20855
section profiling
BytebuddyAgent [baseline] (685.226 ms) : 0, 685226
BytebuddyAgent [candidate] (693.112 ms) : 0, 693112
GlobalTracer [baseline] (432.228 ms) : 0, 432228
GlobalTracer [candidate] (436.128 ms) : 0, 436128
AppSec [baseline] (53.77 ms) : 0, 53770
AppSec [candidate] (54.045 ms) : 0, 54045
Remote Config [baseline] (678.879 µs) : 0, 679
Remote Config [candidate] (660.416 µs) : 0, 660
Telemetry [baseline] (7.667 ms) : 0, 7667
Telemetry [candidate] (7.663 ms) : 0, 7663
ProfilingAgent [baseline] (93.537 ms) : 0, 93537
ProfilingAgent [candidate] (93.533 ms) : 0, 93533
Profiling [baseline] (93.559 ms) : 0, 93559
Profiling [candidate] (93.557 ms) : 0, 93557
LoadParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 13 metrics, 15 unstable metrics. Request duration reports for insecure-bankgantt
title insecure-bank - request duration [CI 0.99] : candidate=1.44.0-SNAPSHOT~e73523127f, baseline=1.44.0-SNAPSHOT~508c314d67
dateFormat X
axisFormat %s
section baseline
no_agent (378.89 µs) : 359, 398
. : milestone, 379,
iast (492.232 µs) : 470, 514
. : milestone, 492,
iast_FULL (663.116 µs) : 641, 685
. : milestone, 663,
iast_GLOBAL (520.846 µs) : 499, 542
. : milestone, 521,
iast_HARDCODED_SECRET_DISABLED (497.425 µs) : 476, 519
. : milestone, 497,
iast_INACTIVE (463.245 µs) : 442, 484
. : milestone, 463,
iast_TELEMETRY_OFF (494.131 µs) : 473, 516
. : milestone, 494,
tracing (465.494 µs) : 444, 487
. : milestone, 465,
section candidate
no_agent (389.279 µs) : 370, 409
. : milestone, 389,
iast (498.615 µs) : 477, 520
. : milestone, 499,
iast_FULL (660.518 µs) : 639, 682
. : milestone, 661,
iast_GLOBAL (519.888 µs) : 499, 541
. : milestone, 520,
iast_HARDCODED_SECRET_DISABLED (500.453 µs) : 479, 522
. : milestone, 500,
iast_INACTIVE (463.597 µs) : 442, 485
. : milestone, 464,
iast_TELEMETRY_OFF (486.172 µs) : 465, 507
. : milestone, 486,
tracing (458.762 µs) : 438, 479
. : milestone, 459,
Request duration reports for petclinicgantt
title petclinic - request duration [CI 0.99] : candidate=1.44.0-SNAPSHOT~e73523127f, baseline=1.44.0-SNAPSHOT~508c314d67
dateFormat X
axisFormat %s
section baseline
no_agent (1.358 ms) : 1338, 1378
. : milestone, 1358,
appsec (1.772 ms) : 1746, 1797
. : milestone, 1772,
appsec_no_iast (1.773 ms) : 1749, 1798
. : milestone, 1773,
iast (1.502 ms) : 1479, 1525
. : milestone, 1502,
profiling (1.511 ms) : 1487, 1535
. : milestone, 1511,
tracing (1.501 ms) : 1477, 1526
. : milestone, 1501,
section candidate
no_agent (1.369 ms) : 1350, 1389
. : milestone, 1369,
appsec (1.767 ms) : 1742, 1793
. : milestone, 1767,
appsec_no_iast (1.753 ms) : 1728, 1778
. : milestone, 1753,
iast (1.511 ms) : 1487, 1534
. : milestone, 1511,
profiling (1.528 ms) : 1505, 1550
. : milestone, 1528,
tracing (1.505 ms) : 1481, 1530
. : milestone, 1505,
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.44.0-SNAPSHOT~e73523127f, baseline=1.44.0-SNAPSHOT~508c314d67
dateFormat X
axisFormat %s
section baseline
no_agent (14.763 s) : 14763000, 14763000
. : milestone, 14763000,
appsec (14.907 s) : 14907000, 14907000
. : milestone, 14907000,
iast (18.98 s) : 18980000, 18980000
. : milestone, 18980000,
iast_GLOBAL (18.033 s) : 18033000, 18033000
. : milestone, 18033000,
profiling (14.998 s) : 14998000, 14998000
. : milestone, 14998000,
tracing (14.916 s) : 14916000, 14916000
. : milestone, 14916000,
section candidate
no_agent (15.617 s) : 15617000, 15617000
. : milestone, 15617000,
appsec (14.992 s) : 14992000, 14992000
. : milestone, 14992000,
iast (18.936 s) : 18936000, 18936000
. : milestone, 18936000,
iast_GLOBAL (17.962 s) : 17962000, 17962000
. : milestone, 17962000,
profiling (14.924 s) : 14924000, 14924000
. : milestone, 14924000,
tracing (15.246 s) : 15246000, 15246000
. : milestone, 15246000,
Execution time for tomcatgantt
title tomcat - execution time [CI 0.99] : candidate=1.44.0-SNAPSHOT~e73523127f, baseline=1.44.0-SNAPSHOT~508c314d67
dateFormat X
axisFormat %s
section baseline
no_agent (1.476 ms) : 1464, 1487
. : milestone, 1476,
appsec (2.362 ms) : 2320, 2403
. : milestone, 2362,
iast (2.098 ms) : 2045, 2151
. : milestone, 2098,
iast_GLOBAL (2.138 ms) : 2085, 2191
. : milestone, 2138,
profiling (1.981 ms) : 1938, 2024
. : milestone, 1981,
tracing (1.94 ms) : 1899, 1981
. : milestone, 1940,
section candidate
no_agent (1.476 ms) : 1465, 1488
. : milestone, 1476,
appsec (2.357 ms) : 2316, 2399
. : milestone, 2357,
iast (2.103 ms) : 2050, 2155
. : milestone, 2103,
iast_GLOBAL (2.15 ms) : 2097, 2204
. : milestone, 2150,
profiling (1.958 ms) : 1916, 2000
. : milestone, 1958,
tracing (1.928 ms) : 1889, 1968
. : milestone, 1928,
|
PerfectSlayer
added a commit
that referenced
this pull request
Nov 28, 2024
amarziali
approved these changes
Nov 28, 2024
smola
approved these changes
Nov 28, 2024
…y configuration file JAVA_TOOL_OPTIONS is limited to 1024 characters and it starts reaching the limit.
PerfectSlayer
force-pushed
the
bbujon/smoke-tests
branch
from
November 28, 2024 17:07
e5ad0f5
to
e735231
Compare
PerfectSlayer
added a commit
that referenced
this pull request
Nov 28, 2024
PerfectSlayer
added a commit
that referenced
this pull request
Nov 29, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
What Does This Do
This PR replaces usage of the
JAVA_TOOL_OPTIONS
environment variable by thejvm.options
server configuration file from OpenLiberty.Motivation
The
JAVA_TOOL_OPTIONS
is limited to 1024 characters on some Java versions and it starts reaching the limit.I also tried
JVM_ARGS
environment variable but it was not picked by our deployment method (runnable jar).Additional Notes
As multiple specifications can use the same runnable jar at the same time (when running tests in parallel), the jar is copied before injecting configuration.
This is a fix required to enable crash-tracking more widely in our system tests #7855
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]