Skip to content

Commit eedb1aa

Browse files
committed
[SPARK-48899][K8S] Fix ENV key value format in K8s Dockerfiles
### What changes were proposed in this pull request? This PR aims to fix `ENV` key value format in K8s Dockerfiles. ### Why are the changes needed? To follow the Docker guideline to fix the following legacy format. - https://docs.docker.com/reference/build-checks/legacy-key-value-format/ ``` - LegacyKeyValueFormat: "ENV key=value" should be used instead of legacy "ENV key value" format ``` ### Does this PR introduce _any_ user-facing change? No. ### How was this patch tested? Pass the CIs. ### Was this patch authored or co-authored using generative AI tooling? No. Closes #47357 from dongjoon-hyun/SPARK-48899. Authored-by: Dongjoon Hyun <dhyun@apple.com> Signed-off-by: Dongjoon Hyun <dhyun@apple.com> (cherry picked from commit 40899c1) Signed-off-by: Dongjoon Hyun <dongjoon@apache.org>
1 parent 3ba36ec commit eedb1aa

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

resource-managers/kubernetes/docker/src/main/dockerfiles/spark/Dockerfile

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -52,7 +52,7 @@ COPY examples /opt/spark/examples
5252
COPY kubernetes/tests /opt/spark/tests
5353
COPY data /opt/spark/data
5454

55-
ENV SPARK_HOME /opt/spark
55+
ENV SPARK_HOME=/opt/spark
5656

5757
WORKDIR /opt/spark/work-dir
5858
RUN chmod g+w /opt/spark/work-dir

resource-managers/kubernetes/docker/src/main/dockerfiles/spark/bindings/R/Dockerfile

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,7 @@ RUN \
3232
rm -rf /var/cache/apt/* && rm -rf /var/lib/apt/lists/*
3333

3434
COPY R ${SPARK_HOME}/R
35-
ENV R_HOME /usr/lib/R
35+
ENV R_HOME=/usr/lib/R
3636

3737
WORKDIR /opt/spark/work-dir
3838
ENTRYPOINT [ "/opt/entrypoint.sh" ]

0 commit comments

Comments
 (0)