kennknowles commented on code in PR #29992: URL: https://github.com/apache/beam/pull/29992#discussion_r1459964873
########## .github/gh-actions-self-hosted-runners/arc/images/Dockerfile: ########## @@ -41,6 +41,9 @@ RUN curl -OL https://cdn.azul.com/zulu/bin/zulu8.70.0.23-ca-jdk8.0.372-linux_x64 tar -C /usr/local -xzf zulu8.70.0.23-ca-jdk8.0.372-linux_x64.tar.gz && \ rm zulu8.70.0.23-ca-jdk8.0.372-linux_x64.tar.gz && \ mv /usr/local/zulu8.70.0.23-ca-jdk8.0.372-linux_x64 /usr/local/java +RUN curl -OL https://cdn.azul.com/zulu/bin/zulu17.46.19-ca-jdk17.0.9-linux_x64.tar.gz && \ + tar -C /usr/local -xzf https://cdn.azul.com/zulu/bin/zulu17.46.19-ca-jdk17.0.9-linux_x64.tar.gz && \ + rm zulu17.46.19-ca-jdk17.0.9-linux_x64.tar.gz Review Comment: OK if it only takes 6 seconds then this isn't a prerequisite for moving ahead with version upgrades. To be clear, this was a step in the process of migrating to 17 as the default for CI, and from there bumping up the minimum version for build so that we could keep up with build tools that have dropped Java 8 support. Aka I think we can commit. -- 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]
