Skip to content

Commit debd1d9

Browse files
committed
[OPENJDK-3038] echo out Tech Preview warning
The jlink feature will initially ship as TP. https://issues.redhat.com/browse/OPENJDK-3038 Signed-off-by: Jonathan Dowland <[email protected]>
1 parent 3dd45b2 commit debd1d9

File tree

3 files changed

+9
-0
lines changed

3 files changed

+9
-0
lines changed

modules/jlink/artifacts/opt/jboss/container/java/jlink/preflight.sh

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,10 @@
1+
jlink_techpreview_warning()
2+
{
3+
echo "Jlink integration is a Tech Preview feature."
4+
echo "See <https://access.redhat.com/support/offerings/techpreview/>"
5+
echo "for more information."
6+
}
7+
18
jlink_preflight_check()
29
{
310
# preflight check: do we have what we need?

modules/jlink/tests/features/jlink.feature

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,7 @@ Scenario: Ensure jlinked builder is used to build the containerized application
77
| S2I_ENABLE_JLINK | true |
88
| QUARKUS_PACKAGE_TYPE| uber-jar |
99
Then run ls /tmp/jre in container and check its output for bin
10+
And container log should contain Jlink integration is a Tech Preview feature.
1011

1112
Scenario: Ensure S2I_ENABLE_JLINK is not set to true
1213
Given s2i build https://github.com/jboss-container-images/openjdk-test-applications from quarkus-quickstarts/getting-started-3.0.1.Final-nos2i

modules/s2i/bash/artifacts/usr/local/s2i/assemble

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -22,6 +22,7 @@ setup_java_app_and_lib
2222
if [ "$S2I_ENABLE_JLINK" = "true" ]; then
2323

2424
source "${JBOSS_CONTAINER_JAVA_JLINK_MODULE}/preflight.sh"
25+
jlink_techpreview_warning
2526
jlink_preflight_check
2627

2728
source "${JBOSS_CONTAINER_JAVA_JLINK_MODULE}/mkdeps.sh"

0 commit comments

Comments
 (0)