fix: set the USER for the blsq image; remove g+w on the wrap_up.py & …

…merge install.packages together
BLSQ · Nov 20, 2024 · 456febf · 456febf
1 parent 80d4e6c
commit 456febf
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 5 deletions.
diff --git a/images/base/files/scripts/fuse_mount.py b/images/base/files/scripts/fuse_mount.py
@@ -47,8 +47,10 @@ def do_GET(self):
     args = [
         "gcsfuse",
         "-o",
-        "allow_other",
+        "rw",
         "--implicit-dirs",  # Also create implicit directories structure (i.e. key /a/b/c/d will create /a, /a/b, /a/b/c)
+        "--uid=1000",  # jovyan user id
+        "--gid=100",  # users group id
     ]
 
     # Use the custom token server to get the token

diff --git a/images/base/files/scripts/wrap_up.py b/images/base/files/scripts/wrap_up.py
@@ -7,5 +7,4 @@
 # subprocess.run(["chmod", "a-w", "/home/jovyan"])
 
 # Make /home/jovyan/worskpace + /home/jovyan/tmp writable
-subprocess.run(["chmod", "g+w", "/home/jovyan/workspace"])
 subprocess.run(["chmod", "g+w", "/home/jovyan/tmp"])
diff --git a/images/blsq/Dockerfile b/images/blsq/Dockerfile
@@ -10,8 +10,7 @@ USER root
 RUN apt-get update && apt-get install -y libgdal-dev libudunits2-dev
 
 # R packages (only available on cran)
-RUN R -e "install.packages('GISTools', dependencies=TRUE, quiet=TRUE, repos='https://cran.r-project.org/')" && \
-    R -e "install.packages('OpenStreetMap', dependencies=TRUE, quiet=TRUE, repos='https://cran.r-project.org/')"
+RUN R -e "options(Ncpus = parallel::detectCores()); install.packages(c('GISTools', 'OpenStreetMap'), dependencies=TRUE, quiet=TRUE, repos='https://cran.r-project.org/', Ncpus=parallel::detectCores())"
 
 # R packages on R channel
 RUN conda install --yes -c R \
@@ -80,4 +79,5 @@ RUN conda install --yes -c conda-forge  -c R\
   conda clean --all -y
 
 # Set PYTHONUNBUFFERED to ensure that Python output is sent straight to terminal
-ENV PYTHONUNBUFFERED=1
+ENV PYTHONUNBUFFERED=1
+USER ${NB_UID}