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

Update docker runtime with the latest openjdk:24-slim image tag #1138

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

vipulg13
Copy link
Contributor

@vipulg13 vipulg13 commented Jul 5, 2024

the openjdk:17-slim image is outdated with high- and critical-level vulnerabilities. Updating it with the latest image should fix the issue.

@lfoppiano lfoppiano added this to the 0.8.2 milestone Jul 6, 2024
@lfoppiano lfoppiano deleted the branch kermitt2:master July 14, 2024 15:09
@lfoppiano lfoppiano closed this Jul 14, 2024
@lfoppiano lfoppiano reopened this Jul 17, 2024
@lfoppiano lfoppiano changed the base branch from fix-vulnerabilities to master July 17, 2024 13:09
@lfoppiano lfoppiano changed the title build runtime with the latest openjdk:24-slim image tag Update docker runtime with the latest openjdk:24-slim image tag Jul 17, 2024
@lfoppiano
Copy link
Collaborator

It think is better to migrate to ecliipse-temurin, no matter which version, will be much better supported and maintained.

@kermitt2
Copy link
Owner

Careful, we had an eclipse-temurin JDK which resulted random crashes with the docker image, I think we need to use an ubuntu official build for the docker image, because the base image for tensorflow is ubuntu-based, and we never had problems with openjdk in contrast.

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.

3 participants