Fix pip install (#85)

* fix: not having jupyter venv in PATH * fix: jupyteruser not owning its venv directory
briefercloud · Sep 17, 2024 · 38e4238 · 38e4238
1 parent fc38ea9
commit 38e4238
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 0 deletions.
diff --git a/docker/Dockerfile b/docker/Dockerfile
@@ -125,6 +125,7 @@ RUN groupadd -r briefer \
     && useradd -r -g jupyteruser jupyteruser \
     && mkdir -p /home/jupyteruser \
     && chown -R jupyteruser:jupyteruser /home/jupyteruser \
+    && chown -R jupyteruser:jupyteruser /app/jupyter/venv \
     && chmod 700 /home/jupyteruser
 
 

diff --git a/docker/setup/jupyter.py b/docker/setup/jupyter.py
@@ -27,6 +27,7 @@ def get_config():
 def run_jupyter(cfg):
     env = os.environ.copy()
     env["JUPYTER_TOKEN"] = cfg["JUPYTER_TOKEN"]
+    env["PATH"] = "/app/jupyter/venv/bin:" + env.get("PATH", "")
 
     jupyter = subprocess.run([
         "/app/jupyter/venv/bin/jupyter",