heesung-sn commented on code in PR #15264:
URL: https://github.com/apache/pulsar/pull/15264#discussion_r858958848


##########
pulsar-client-cpp/docker-tests.sh:
##########
@@ -65,7 +65,10 @@ DISABLE_COLOR_OUTPUT=""
 if [ "$GTEST_COLOR" = "no" ]; then
   DISABLE_COLOR_OUTPUT="| cat"
 fi
-$DOCKER_CMD bash -c "set -o pipefail; cd /pulsar/pulsar-client-cpp && 
./run-unit-tests.sh ${tests} $DISABLE_COLOR_OUTPUT"
+
+# Java17 is required for CLI e.g) bin/pulsar create-token
+$DOCKER_CMD bash -c "apt-get -y install openjdk-17-jre-headless &&\

Review Comment:
   Unfortunately, we currently use this image, 
[apachepulsar/pulsar-build:ubuntu-16.04-pb3](https://hub.docker.com/layers/pulsar-build/apachepulsar/pulsar-build/ubuntu-16.04-pb3/images/sha256-0209721f1acbd335d63141fa5ab1513e1b78d9fc7e36bda125bb7b584ba04609?context=explore),
 which does not have jdk17.
   
   I am happy to create a new image with jdk17 and use it here, but I don't 
think I have a permission to do this. Let me see if someone from SN team can 
help here.
   



-- 
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]

Reply via email to