chia7712 commented on code in PR #20897: URL: https://github.com/apache/kafka/pull/20897#discussion_r2547557055
########## tests/docker/ducker-ak: ########## @@ -44,8 +44,9 @@ docker_run_memory_limit="2000m" # The default number of cluster nodes to bring up if a number is not specified. default_num_nodes=14 -# The default OpenJDK base image. -default_jdk="openjdk:8" +# The default JDK base image with apt-get support. +# The openjdk image has been officially deprecated. For more information, see: https://hub.docker.com/_/openjdk +default_jdk="sapmachine:11-jdk-ubuntu-jammy" Review Comment: Since Java 11 is not supported by older Kafka versions, we must use `eclipse-temurin:8-jdk-jammy` instead. The `PATH` variable presents an issue, which @unknowntpo and I discussed offline. The simplest solution is to create a direct symlink, which eliminate the need to modify config files such as `environment` and `profile` -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
