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

feat: upgrade to Java 17 #1279

Closed
wants to merge 5 commits into from
Closed

Conversation

loicmathieu
Copy link

This PR updates AKHQ to Java 17.

@tchiotludo an alternative can be to update only the container to 17 and build it for both 11 and 17 so compatibility for 11 is kept in case someone deploy it via it's JAR and not a Docker container.

We discuss the possibility to provides both a container with the JRE and the JDK, I didn't do it because I have no idea how your container is built and deployed. May I add a second Dockerfile ?

lucapette and others added 4 commits November 9, 2022 21:47
* Update node to the latest LTS version
* Replace node-sass with sass
* Also use the correct version of react-test-renderer.
* Update react scripts so we don't use insecure hashing functions
* Do not check eslint for production build

See tchiotludo#1243 for more information.
@tchiotludo
Copy link
Owner

The best for the JDK will be to add a build-arg to the docker image with a default to jre 17 and to provide another github action for pushing the image with jdk with another tag.

For the compatibility with 11, I think to force upgrade and to provide only a 17 version for now. Codebase will not use features from 17 for now, and will wait to see if people is complaining on that. Anyway, I'm almost people will complain (when I see people creating ticket with akhq version 0.14 🙄)

@loicmathieu loicmathieu marked this pull request as ready for review November 28, 2022 10:10
@tchiotludo
Copy link
Owner

@loicmathieu you don't add the jdk image?

@loicmathieu
Copy link
Author

@tchiotludo no, I don't know how to do it exactly and don't have time now to investiguate.

If you want I can provide a separate PR for it but you need to explain to me how you make the release, as fas as I understand the workflow tags the images based on GIT tags. As there will not be different GIT tags for JRE vs JDK images more tags should be added manually right ?

@tchiotludo
Copy link
Owner

done in #1615

@tchiotludo tchiotludo closed this Dec 6, 2023
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.

5 participants