Skip to content

Commit 62f47eb

Browse files
authored
Merge pull request #82 from jmesnil/eap81_jdk21_images
Support JDK21 images for S2I
2 parents 641da34 + 97bdd49 commit 62f47eb

File tree

4 files changed

+31
-12
lines changed

4 files changed

+31
-12
lines changed

charts/eap81/README.md

+3-1
Original file line numberDiff line numberDiff line change
@@ -137,9 +137,11 @@ If the application image has been built by another mechanism, you can skip the b
137137
| `build.s2i.galleonDir` | Directory relative to the root directory for the build that contains custom content for Galleon. | - | [EAP Documentation](https://access.redhat.com/documentation/en-us/red_hat_jboss_enterprise_application_platform/7.4/html/getting_started_with_jboss_eap_for_openshift_container_platform/index) |
138138
| `build.s2i.featurePacks` | List of additional Galleon feature-packs identified by Maven coordinates (`<groupId>:<artifactId>:<version>`) | - | The value can be be either a `string` with a list of comma-separated Maven coordinate or an array where each item is the Maven coordinate of a feature pack - [EAP Documentation](https://access.redhat.com/documentation/en-us/red_hat_jboss_enterprise_application_platform/7.4/html/getting_started_with_jboss_eap_for_openshift_container_platform/index) |
139139
| `build.s2i.channels` | List of Channels identified by Maven coordinates (`<groupId>:<artifactId>`). If featurePacks are configured without any versioning, the channels that provides the latest feature packs can be specified. Deprecated, the recommended way to provision EAP is to use the eap-maven-plugin in the application pom.xml | - | The value can be be either a `string` with a list of comma-separated Maven coordinate or an array where each item is the Maven coordinate of a channel - [EAP Documentation](https://access.redhat.com/documentation/en-us/red_hat_jboss_enterprise_application_platform/7.4/html/getting_started_with_jboss_eap_for_openshift_container_platform/index) |
140-
| `build.s2i.jdk` | JDK Version of the EAP S2I images | `"17"` | Allowed Values: `"17"` |
140+
| `build.s2i.jdk` | JDK Version of the EAP S2I images | `"17"` | Allowed Values: `"17", "21"` |
141141
| `build.s2i.jdk17.builderImage` | EAP S2I Builder image for JDK 17 | `registry.redhat.io/jboss-eap-8-tech-preview/eap81-openjdk17-builder-openshift-rhel9:latest` | [EAP Documentation](https://access.redhat.com/documentation/en-us/red_hat_jboss_enterprise_application_platform/8.1/html/getting_started_with_jboss_eap_for_openshift_container_platform/index) |
142142
| `build.s2i.jdk17.runtimeImage` | EAP S2I Runtime image for JDK 17| `registry.redhat.io/jboss-eap-8-tech-preview/eap81-openjdk17-runtime-openshift-rhel9:latest` | [EAP Documentation](https://access.redhat.com/documentation/en-us/red_hat_jboss_enterprise_application_platform/8.1/html/getting_started_with_jboss_eap_for_openshift_container_platform/index) |
143+
| `build.s2i.jdk21.builderImage` | EAP S2I Builder image for JDK 21 | `registry.redhat.io/jboss-eap-8-tech-preview/eap81-openjdk21-builder-openshift-rhel9:latest` | [EAP Documentation](https://access.redhat.com/documentation/en-us/red_hat_jboss_enterprise_application_platform/8.1/html/getting_started_with_jboss_eap_for_openshift_container_platform/index) |
144+
| `build.s2i.jdk21.runtimeImage` | EAP S2I Runtime image for JDK 21| `registry.redhat.io/jboss-eap-8-tech-preview/eap81-openjdk21-runtime-openshift-rhel9:latest` | [EAP Documentation](https://access.redhat.com/documentation/en-us/red_hat_jboss_enterprise_application_platform/8.1/html/getting_started_with_jboss_eap_for_openshift_container_platform/index) |
143145
| `build.s2i.kind` | Determines the type of images for S2I Builder and Runtime images (`DockerImage`, `ImageStreamTag` or `ImageStreamImage`) | `DockerImage` | [OKD Documentation](https://docs.okd.io/latest/cicd/builds/build-strategies.html#builds-strategy-s2i-build_build-strategies) |
144146
| `build.s2i.runtimeKind` | Determines the type of images for S2I Runtime image (`DockerImage`, `ImageStreamTag` or `ImageStreamImage`) | the value of `build.s2i.kind` | [OKD Documentation](https://docs.okd.io/latest/cicd/)|
145147
| `build.sourceSecret`|Name of the secret containing the credentials to login to Git source reposiory | - | The secret must exist in the same namespace or the chart will fail to install - [OKD documentation](https://docs.okd.io/latest/cicd/builds/creating-build-inputs.html#builds-manually-add-source-clone-secrets_creating-build-inputs) |

charts/eap81/templates/_helpers.tpl

+8-8
Original file line numberDiff line numberDiff line change
@@ -4,10 +4,10 @@ eap8.eapBuilderImage corresponds to the imagestream for the EAP S2I Builder imag
44
It depends on the build.s2i.jdk version.
55
*/}}
66
{{- define "eap8.eapBuilderImage" -}}
7-
{{- if eq .Values.build.s2i.jdk "11" -}}
8-
{{ .Values.build.s2i.jdk11.builderImage}}
9-
{{- else -}}
7+
{{- if eq .Values.build.s2i.jdk "17" -}}
108
{{ .Values.build.s2i.jdk17.builderImage}}
9+
{{- else -}}
10+
{{ .Values.build.s2i.jdk21.builderImage}}
1111
{{- end -}}
1212
{{- end -}}
1313

@@ -16,10 +16,10 @@ eap8.eapRuntimeImage corresponds to the imagestream for the EAP S2I Runtime imag
1616
It depends on the build.s2i.jdkVersion.
1717
*/}}
1818
{{- define "eap8.eapRuntimeImage" -}}
19-
{{- if eq .Values.build.s2i.jdk "11" -}}
20-
{{ .Values.build.s2i.jdk11.runtimeImage}}
21-
{{- else -}}
19+
{{- if eq .Values.build.s2i.jdk "17" -}}
2220
{{ .Values.build.s2i.jdk17.runtimeImage}}
21+
{{- else -}}
22+
{{ .Values.build.s2i.jdk21.runtimeImage}}
2323
{{- end -}}
2424
{{- end -}}
2525

@@ -40,9 +40,9 @@ app.openshift.io/runtime: eap
4040
{{- define "eap8.metering.labels" -}}
4141
com.company: "Red_Hat"
4242
rht.prod_name: "Red_Hat_Runtimes"
43-
rht.prod_ver: "2023-Q4"
43+
rht.prod_ver: "2025-Q2"
4444
rht.comp: "EAP"
45-
rht.comp_ver: "8.0"
45+
rht.comp_ver: "8.1"
4646
rht.subcomp_t: "application"
4747
{{- end }}
4848

charts/eap81/values.schema.json

+17-3
Original file line numberDiff line numberDiff line change
@@ -234,7 +234,7 @@
234234
"jdk": {
235235
"description": "JDK Version of the EAP S2I images",
236236
"type": "string",
237-
"enum": ["17"],
237+
"enum": ["17", "21"],
238238
"default": "17"
239239
},
240240
"jdk17": {
@@ -248,9 +248,23 @@
248248
"runtimeImage": {
249249
"description": "EAP S2I Runtime image for JDK 17",
250250
"type": "string"
251-
}
252251
}
253-
},
252+
}
253+
},
254+
"jdk21": {
255+
"description": "EAP S2I images for JDK 21",
256+
"type": "object",
257+
"properties": {
258+
"builderImage": {
259+
"description": "EAP S2I Builder image for JDK 21",
260+
"type": "string"
261+
},
262+
"runtimeImage": {
263+
"description": "EAP S2I Runtime image for JDK 21",
264+
"type": "string"
265+
}
266+
}
267+
},
254268
"buildApplicationImage": {
255269
"description": "Determine if the application image must be built. If false, the Helm release will build the first artifact image (with the name of the Helm release)",
256270
"type": "boolean",

charts/eap81/values.yaml

+3
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,9 @@ build:
1515
jdk17:
1616
builderImage: registry.redhat.io/jboss-eap-8-tech-preview/eap81-openjdk17-builder-openshift-rhel9:latest
1717
runtimeImage: registry.redhat.io/jboss-eap-8-tech-preview/eap81-openjdk17-runtime-openshift-rhel9:latest
18+
jdk21:
19+
builderImage: registry.redhat.io/jboss-eap-8-tech-preview/eap81-openjdk21-builder-openshift-rhel9:latest
20+
runtimeImage: registry.redhat.io/jboss-eap-8-tech-preview/eap81-openjdk21-runtime-openshift-rhel9:latest
1821
output:
1922
kind: "ImageStreamTag"
2023
triggers: {}

0 commit comments

Comments
 (0)