[DH-394] just a couple of minor tweaks

berkeley-dsep-infra · Oct 10, 2024 · ae0d3c7 · ae0d3c7
1 parent 19084d5
commit ae0d3c7
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 3 deletions.
diff --git a/Dockerfile b/Dockerfile
@@ -54,8 +54,8 @@ RUN apt-get update > /dev/null && \
 COPY install-mambaforge.bash /tmp/install-mambaforge.bash
 RUN /tmp/install-mambaforge.bash
 
-RUN chown -Rh rstudio:rstudio /home/rstudio
-RUN chown -Rh rstudio:rstudio /srv/conda
+# needed for building on mac see DH-394
+RUN chown -Rh ${NB_USER}:${NB_USER} ${HOME}
 
 USER ${NB_USER}
 
@@ -92,4 +92,8 @@ RUN r /tmp/r-packages/2021-spring-phw-272a.r
 # Use simpler locking strategy
 COPY file-locks /etc/rstudio/file-locks
 
+# Doing a little cleanup
+RUN rm -rf /tmp/downloaded_packages
+RUN rm -rf ${HOME}/.cache
+
 ENTRYPOINT ["tini", "--"]
diff --git a/install-mambaforge.bash b/install-mambaforge.bash
@@ -37,6 +37,6 @@ rm ${INSTALLER_PATH}
 # Remove the pip cache created as part of installing mambaforge
 rm -rf /root/.cache
 
-chown -R $NB_USER:$NB_USER ${CONDA_DIR}
+chown -Rh $NB_USER:$NB_USER ${CONDA_DIR}
 
 conda list -n root