Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

start.sh: JUPYTER_ENV_VARS_TO_UNSET introduced #1524

Merged
merged 4 commits into from
Nov 15, 2021

Conversation

maresb
Copy link
Contributor

@maresb maresb commented Nov 11, 2021

Cherry-pick of b44b7ab. I'm working through #1512 (comment)

Allow you to define variables to be unset before running the command
that start.sh is supposed to start. These variables will still be
available in the hooks run before.

Allow you to define variables to be unset before running the command
that start.sh is supposed to start. These variables will still be
available in the hooks run before.
base-notebook/start.sh Outdated Show resolved Hide resolved
base-notebook/test/test_container_options.py Show resolved Hide resolved
@maresb
Copy link
Contributor Author

maresb commented Nov 12, 2021

Feel free to commit to my branch, since I won't be able to work on this for a while. I mainly wanted to cherry-pick the remaining commits into separate PRs before I forgot how everything went.

Copy link
Member

@mathbunnyru mathbunnyru left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We should definitely cover the non-root user with a test since that's a separate execution thread in the script

Only this one is left here.

base-notebook/test/test_container_options.py Show resolved Hide resolved
@mathbunnyru
Copy link
Member

I added non-root case, let's hope it's gonna work.

@mathbunnyru mathbunnyru merged commit 5cb44a8 into jupyter:master Nov 15, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants