Skip to content

Commit

Permalink
Remove NB_UID ENV command
Browse files Browse the repository at this point in the history
The NB_UID environment variable doesn't need to be set anymore as we are
handling the UID for the running user in the corresponding helm chart now. This
is better because setting the NB_UID in the docker file causes the upstream
image to `chown` the home directory to this uid, but this is slow and gets
slower as the users have more files in their home directory.

We are now making the cluster launch the pod with the running user as 1001 so
this chown is not required
  • Loading branch information
r4vi committed Nov 29, 2018
1 parent 6064ffe commit 302af1b
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 3 deletions.
1 change: 0 additions & 1 deletion all-spark-notebook/Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,6 @@ LABEL [email protected]
USER root

ENV PATH=$PATH:$HOME/.local/bin
ENV NB_UID=1001

ENV PYSPARK_SUBMIT_ARGS="--packages com.amazonaws:aws-java-sdk:1.7.4,org.apache.hadoop:hadoop-aws:2.7.1 pyspark-shell"

Expand Down
2 changes: 0 additions & 2 deletions datascience-notebook/Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -6,8 +6,6 @@ USER root

ENV PATH=$PATH:$HOME/.local/bin

ENV NB_UID=1001

COPY ./files/apt_packages /tmp/

RUN conda install boto3 \
Expand Down

0 comments on commit 302af1b

Please sign in to comment.