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

jupyterhub 0.9.2 #160

Merged
merged 2 commits into from Aug 29, 2018
Merged

jupyterhub 0.9.2 #160

merged 2 commits into from Aug 29, 2018

Conversation

minrk
Copy link
Member

@minrk minrk commented Aug 28, 2018

fixes TERM shutdown

bumps oauthenticator to 0.8 as well

fixes TERM shutdown
leaves users running when the hub restarts

Tests were relying on a bug in 0.9.1 that prevented cleanup due to improper handling of signals
@minrk
Copy link
Member Author

minrk commented Aug 29, 2018

Turns out we were actually relying on the bug in 0.9.1, where cleanup wouldn't fire due to an error in the SIGTERM handler. This is fixed by setting c.JupyterHub.cleanup_servers = False in config.

@yuvipanda yuvipanda merged commit 91b405d into jupyterhub:master Aug 29, 2018
@yuvipanda
Copy link
Collaborator

Yay tests that catch things!!!! <3

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.

None yet

2 participants