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

[JBEAP-24939] JDK 17 Support for XP 4 Helm Charts #60

Merged
merged 1 commit into from
Sep 28, 2023

Conversation

sudeshnas93
Copy link
Contributor

@sudeshnas93 sudeshnas93 commented Sep 5, 2023

Issue: https://issues.redhat.com/browse/JBEAP-24939

Note: Please merge #61 before merging this issue.

@github-actions github-actions bot added charts documentation Improvements or additions to documentation eap-xp4 labels Sep 5, 2023
| `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) |
| `build.bootableJar.builderImage` | JDK Builder image for Bootable Jar | [registry.access.redhat.com/ubi8/openjdk-11:latest](https://catalog.redhat.com/software/containers/ubi8/openjdk-11/5dd6a4b45a13461646f677f4?gti-tabs=unauthenticated) | - |
| `build.bootableJar.builderImage` | JDK Builder image for Bootable Jar | [registry.access.redhat.com/ubi8/openjdk-11:latest](https://catalog.redhat.com/software/containers/ubi8/openjdk-11/5dd6a4b45a13461646f677f4?gti-tabs=unauthenticated) | - |
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It seems there's a formatting issue on your IDE that's adding spaces after end on line?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, I am using intellij idea ultimate latest version. But whenever I am trying to put a space or special character like comma after " Allowed Values: "11" ", a new line gets generated in the table. Although, I can confirm that the preview of the table looks fine to me.
I am still trying to remove this formatting issue and will send an updated change soon.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You can format it using Ctrl+Alt+L or use a plugin pipe-table-formatter and For trailing spaces use Settings → Editor → General → Strip trailing spaces

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @RanabirChakraborty . Although, these didn't work; I had to manually change it in vim.

@sudeshnas93
Copy link
Contributor Author

This PR is dependent on #61. Please merge #61 before #60.

@sudeshnas93 sudeshnas93 marked this pull request as ready for review September 6, 2023 12:55
@@ -77,7 +77,7 @@ The configuration to build the application image is configured in a `build` sect
| `build.triggers.genericSecret`| Name of the secret containing the WebHookSecretKey for the Generic Webhook | - | The secret must exist in the same namespace or the chart will fail to install - [OpenShift documentation](https://docs.openshift.com/container-platform/latest/cicd/builds/triggering-builds-build-hooks.html#builds-webhook-triggers_triggering-builds-build-hooks) |
| `build.s2i` | Configuration specific to building with EAP S2I images | - | - |
| `build.s2i.version` | Version of the EAP S2I images | Defaults to this chart `AppVersion` | - |
| `build.s2i.jdk` | JDK Version of the EAP S2I images | `"11"` | Allowed Values: `"11"` |
| `build.s2i.jdk` | JDK Version of the EAP S2I images | `"11"` | Allowed Values: `"11"`, `"17"` |
| `build.s2i.jdk11.builderImage` | EAP S2I Builder image for JDK 11 | `registry.redhat.io/jboss-eap-7/eap-xp4-openjdk11-openshift-rhel8` | [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) |
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should there be a JDK 17 equivalent?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Added the JDK17 equivalent images. Thanks.

@sudeshnas93
Copy link
Contributor Author

Hello Jeff Mesnil ( @jmesnil ) ,
Could you please review this PR?
Thanks.

@jmesnil jmesnil merged commit 8b2e1e1 into jbossas:main Sep 28, 2023
2 checks passed
@jmesnil
Copy link
Contributor

jmesnil commented Sep 28, 2023

@sudeshnas93 thanks!

@sudeshnas93
Copy link
Contributor Author

@sudeshnas93 thanks!

Hello Jeff Mesnil( @jmesnil ),
I would request you to merge #61 else this PR won't work as expected. Thanks :)

@jmesnil jmesnil added this to the EAP XP4 1.1.0 milestone Sep 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
charts documentation Improvements or additions to documentation eap-xp4
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants