PDavid commented on code in PR #142: URL: https://github.com/apache/hbase-thirdparty/pull/142#discussion_r2225552542
########## dev-support/jenkins/Dockerfile: ########## @@ -14,21 +14,112 @@ # See the License for the specific language governing permissions and # limitations under the License. -# Dockerfile for hbase-operator-tools pre-commit build. -# https://builds.apache.org/job/PreCommit-HBASE-OPERATOR-TOOLS-Build - -FROM maven:3.9-eclipse-temurin-8 - -# hadolint ignore=DL3008 -RUN apt-get -q update && apt-get -q install --no-install-recommends -y \ - binutils \ - git \ - rsync \ - shellcheck \ - patch \ - wget && \ +# Dockerfile for hbase-thirdparty pre-commit build. +# +# +# Built in multiple stages so as to avoid re-downloading large binaries when +# tweaking unrelated aspects of the image. +FROM ubuntu:22.04 AS base_image Review Comment: Just out of curiosity: Is there a specific reason why we would like to use 22.04 'Jammy Jellyfish' (LTS)? It is a bit older Ubuntu version, we might consider newer version, for example: 24.04 'Noble Numbat' (LTS). Checking at https://hub.docker.com/_/ubuntu 22.04 is also a supported version though. -- 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: issues-unsubscr...@hbase.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
