mingyen066 commented on code in PR #20897: URL: https://github.com/apache/kafka/pull/20897#discussion_r2548426372
########## 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: Thanks, I've updated the changes accordingly. I've verified the fix locally on macOS by running pluggable_test. The latest description and test results have been updated in the PR description. ########## 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: Thanks, I've updated the changes accordingly. I've verified the fix locally on macOS by running `pluggable_test`. The latest description and test results have been updated in the PR description. -- 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]
