chia7712 commented on code in PR #20897: URL: https://github.com/apache/kafka/pull/20897#discussion_r2548539568
########## 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: There are three solutions that should be compatible with most linux distributions: 1. add a symlink for java binary to `/usr/bin` 2. disable PAM from reading `/etc/environment`, which prevents the `PATH` from being overridden 3. rewrite `/etc/environment` during the docker image build process I prefer option 1 because it is straightforward, and the side effect - the inability to easily switch the java version - is acceptable for e2e testing since we don't have that specific use case -- 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]
