Skip to content

Commit

Permalink
[BUG] Removing jovyan home directory and issue with nss confiuration (#…
Browse files Browse the repository at this point in the history
  • Loading branch information
costrouc authored Mar 3, 2022
1 parent c0a49ea commit 3470504
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 6 deletions.
8 changes: 6 additions & 2 deletions .github/workflows/kubernetes_test.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -196,10 +196,14 @@ jobs:
- name: Cleanup qhub deployment
run: |
cd local-deployment
# This is a known failure at the moment.
qhub destroy --config qhub-config.yaml || true
qhub destroy --config qhub-config.yaml
- name: Basic kubectl checks after cleanup
if: always()
run: |
kubectl get all,cm,secret,ing -A
- name: Delete minikube cluster
if: always()
run: |
minikube delete
3 changes: 0 additions & 3 deletions qhub/template/image/scripts/install-conda.sh
Original file line number Diff line number Diff line change
Expand Up @@ -30,9 +30,6 @@ auto_update_conda: false
aggressive_update_packages: []
envs_dirs:
- /home/conda/environments
- /home/jovyan/.conda/envs
pkgs_dirs:
- /home/jovyan/.conda/pkgs
EOF

# Fix permissions in accordance with jupyter stack permissions
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -386,7 +386,7 @@ def deep_merge(d1, d2):

def generate_nss_files(users, groups):
etc_passwd = []
passwd_format = "{username}:x:{uid}:{gid}:{username}:/home/jovyan:/bin/bash"
passwd_format = "{username}:x:{uid}:{gid}:{username}:/home/{username}:/bin/bash"
for user in users:
etc_passwd.append(passwd_format.format(**user))

Expand Down

0 comments on commit 3470504

Please sign in to comment.