e0a80f2650
# 🔍 Description ## Issue References 🔗 This pull request fixes # ## Describe Your Solution 🔧 - fix the dockerfile warning reported by `GitHub Actions / Kyuubi Server On Kubernetes Integration Test` : - Legacy key/value format with whitespace separator should not be used LegacyKeyValueFormat: "ENV key=value" should be used instead of legacy "ENV key value" format More info: https://docs.docker.com/go/dockerfile/rule/legacy-key-value-format/ - The 'as' keyword should match the case of the 'from' keyword FromAsCasing: 'as' and 'FROM' keywords' casing do not match More info: https://docs.docker.com/go/dockerfile/rule/from-as-casing/ ## Types of changes 🔖 - [ ] Bugfix (non-breaking change which fixes an issue) - [ ] New feature (non-breaking change which adds functionality) - [ ] Breaking change (fix or feature that would cause existing functionality to change) ## Test Plan 🧪 #### Behavior Without This Pull Request ⚰️ #### Behavior With This Pull Request 🎉 #### Related Unit Tests --- # Checklist 📝 - [ ] This patch was not authored or co-authored using [Generative Tooling](https://www.apache.org/legal/generative-tooling.html) **Be nice. Be informative.** Closes #6762 from bowenliang123/docker-warning. Closes #6762 d9d7b7465 [Bowen Liang] cleanup dockerfile warning Authored-by: Bowen Liang <liangbowen@gf.com.cn> Signed-off-by: Bowen Liang <liangbowen@gf.com.cn>
90 lines
2.8 KiB
Docker
90 lines
2.8 KiB
Docker
#
|
|
# Licensed to the Apache Software Foundation (ASF) under one or more
|
|
# contributor license agreements. See the NOTICE file distributed with
|
|
# this work for additional information regarding copyright ownership.
|
|
# The ASF licenses this file to You under the Apache License, Version 2.0
|
|
# (the "License"); you may not use this file except in compliance with
|
|
# the License. You may obtain a copy of the License at
|
|
#
|
|
# http://www.apache.org/licenses/LICENSE-2.0
|
|
#
|
|
# Unless required by applicable law or agreed to in writing, software
|
|
# distributed under the License is distributed on an "AS IS" BASIS,
|
|
# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
|
|
# See the License for the specific language governing permissions and
|
|
# limitations under the License.
|
|
#
|
|
|
|
# This Dockerfile is for CI testing purpose, not for end-users
|
|
|
|
# Usage:
|
|
# Run the docker command below
|
|
# docker build \
|
|
# --build-arg MVN_ARG="-Pspark-3.5" \
|
|
# --file build/Dockerfile.CI \
|
|
# --tag apache/kyuubi:<tag> \
|
|
# .
|
|
# Options:
|
|
# -f, --file this docker file
|
|
# -t, --tag the target repo and tag name
|
|
# more options can be found with -h, --help
|
|
|
|
# Declare the BASE_IMAGE argument in the first line, for more detail
|
|
# see: https://github.com/moby/moby/issues/38379
|
|
ARG BASE_IMAGE=eclipse-temurin:8-jdk-focal
|
|
|
|
FROM eclipse-temurin:8-jdk-focal AS builder
|
|
|
|
ARG MVN_ARG
|
|
|
|
# Pass the environment variable `CI` into container, for internal use only.
|
|
#
|
|
# Continuous integration(aka. CI) services like GitHub Actions always provide
|
|
# an environment variable `CI` in runners, and we detect this variable to run some
|
|
# specific actions, e.g. run `mvn` in batch mode to suppress noisy logs.
|
|
ARG CI
|
|
ENV CI=${CI}
|
|
|
|
ADD . /workspace/kyuubi
|
|
WORKDIR /workspace/kyuubi
|
|
|
|
RUN apt-get update && \
|
|
DEBIAN_FRONTEND=noninteractive \
|
|
apt-get install -y bash python3 && \
|
|
ln -snf /bin/bash /bin/sh && \
|
|
./build/dist ${MVN_ARG} && \
|
|
mv /workspace/kyuubi/dist /opt/kyuubi && \
|
|
# Removing stuff saves time because docker creates a temporary layer
|
|
rm -rf ~/.m2 && \
|
|
rm -rf /workspace/kyuubi
|
|
|
|
FROM ${BASE_IMAGE}
|
|
|
|
ARG kyuubi_uid=10009
|
|
|
|
USER root
|
|
|
|
ENV KYUUBI_HOME=/opt/kyuubi
|
|
ENV KYUUBI_LOG_DIR=${KYUUBI_HOME}/logs
|
|
ENV KYUUBI_PID_DIR=${KYUUBI_HOME}/pid
|
|
ENV KYUUBI_WORK_DIR_ROOT=${KYUUBI_HOME}/work
|
|
|
|
COPY --from=builder /opt/kyuubi ${KYUUBI_HOME}
|
|
|
|
RUN set -ex && \
|
|
apt-get update && \
|
|
DEBIAN_FRONTEND=noninteractive \
|
|
apt-get install -y bash tini libc6 libpam-modules krb5-user libnss3 procps && \
|
|
ln -snf /bin/bash /bin/sh && \
|
|
useradd -u ${kyuubi_uid} -g root kyuubi && \
|
|
mkdir -p ${KYUUBI_HOME} ${KYUUBI_LOG_DIR} ${KYUUBI_PID_DIR} ${KYUUBI_WORK_DIR_ROOT} && \
|
|
chmod ug+rw -R ${KYUUBI_HOME} && \
|
|
chmod a+rwx -R ${KYUUBI_WORK_DIR_ROOT} && \
|
|
rm -rf /var/cache/apt/*
|
|
|
|
WORKDIR ${KYUUBI_HOME}
|
|
|
|
CMD [ "./bin/kyuubi", "run" ]
|
|
|
|
USER ${kyuubi_uid}
|