Skip to content

Commit

Permalink
Fix style
Browse files Browse the repository at this point in the history
  • Loading branch information
mathbunnyru committed Aug 25, 2023
1 parent 04b3d0c commit 6aded4b
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 2 deletions.
3 changes: 2 additions & 1 deletion images/docker-stacks-foundation/start.sh
Original file line number Diff line number Diff line change
Expand Up @@ -136,8 +136,8 @@ if [ "$(id -u)" == 0 ] ; then
# NOTE: This hook is run as the root user!
# shellcheck disable=SC1091
source /usr/local/bin/run-hooks.sh /usr/local/bin/before-notebook.d

unset_explicit_env_vars

_log "Running as ${NB_USER}:" "${cmd[@]}"
exec sudo --preserve-env --set-home --user "${NB_USER}" \
LD_LIBRARY_PATH="${LD_LIBRARY_PATH}" \
Expand Down Expand Up @@ -234,6 +234,7 @@ else
# shellcheck disable=SC1091
source /usr/local/bin/run-hooks.sh /usr/local/bin/before-notebook.d
unset_explicit_env_vars

_log "Executing the command:" "${cmd[@]}"
exec "${cmd[@]}"
fi
2 changes: 1 addition & 1 deletion tests/docker-stacks-foundation/test_user_options.py
Original file line number Diff line number Diff line change
Expand Up @@ -243,7 +243,7 @@ def test_container_not_delete_bind_mount(


@pytest.mark.parametrize("enable_root", [False, True])
def test_jupyter_env_vars_to_unset_as_root(
def test_jupyter_env_vars_to_unset(
container: TrackedContainer, enable_root: bool
) -> None:
"""Environment variables names listed in JUPYTER_ENV_VARS_TO_UNSET
Expand Down

0 comments on commit 6aded4b

Please sign in to comment.