Skip to content
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

[Openjdk-2969] runtime images and JAVA_ARGS, JAVA_OPTIONS handling (ubi8) #492

Draft
wants to merge 5 commits into
base: ubi8
Choose a base branch
from

Conversation

jmtd
Copy link
Member

@jmtd jmtd commented May 10, 2024

Some of the run logic is implemented in bin/s2i which means it is not applied in the runtime images.

https://issues.redhat.com/browse/OPENJDK-2969

jmtd added 4 commits May 10, 2024 14:30
Passes for current builders, fails for current runtimes.

https://issues.redhat.com/browse/OPENJDK-2968

Signed-off-by: Jonathan Dowland <[email protected]>
Move the feature file with the JAVA_OPTIONS test into a module
which is included by the runtime images (run) and retag it so
it is applied to the runtime images. Improve the test's log
check.

Test passes for builder images and fails for runtime images.

https://issues.redhat.com/browse/OPENJDK-2968

Signed-off-by: Jonathan Dowland <[email protected]>
This was used in the RHEL7 images but not since.

Signed-off-by: Jonathan Dowland <[email protected]>
@jmtd jmtd changed the title [Openjdk-2968] runtime images and JAVA_ARGS, JAVA_OPTIONS handling [Openjdk-2969] runtime images and JAVA_ARGS, JAVA_OPTIONS handling (ubi8) May 10, 2024
Move the handling of JAVA_OPTIONS, JAVA_OPTS and suffixing of
JAVA_ARGS onto the command line, as well as dispatching to an
application-supplied run script, from s2i/run to run-java.sh.

This is to make this logic available to the runtime images.

<https://issues.redhat.com/browse/OPENJDK-2968>

Signed-off-by: Jonathan Dowland <[email protected]>
@jmtd jmtd force-pushed the OPENJDK-2968-JAVA_ARGS_runtime branch from 5d17ddd to b08c9fc Compare May 10, 2024 20:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant