-
Notifications
You must be signed in to change notification settings - Fork 5k
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
z2h/notebook 7.0.0rc2 users redirected to tree
even though defaultUrl
is set to lab
#6954
Comments
Pinging @jtpio here. The PR may have caused a regression. |
Thanks @shaneknapp for reporting. #6941 does look related indeed. However it also fixed #6940. So not sure the change could just be reverted as it will very likely bring #6940 back. |
Maybe related, the base-notebook image which is configured to run lab, defaults the url to /tree as well, which is unexpected. |
Yup, we spent a good part of our day here trying to track down what was going on with this. This is indeed very unexpected behavior. |
Looking into this in #6997. |
Description
basic environment: jupyterhub 4.0.1/jupyterlab 4.0.2/jupyter-notebook 7.0.0rc2
i followed the z2h instructions to set the default environment to jupyterlab.
here is that relevant bit in our config: https://github.com/berkeley-dsep-infra/datahub/blob/staging/deployments/data100-jl4/config/common.yaml#L41
when logging in, however, all users are by default sent to
tree
:you can only access lab via the drop down menu:
checking the logs (
kubectl -n data100-jl4-staging logs -f jupyter-sknapp
), i see thatdefaultUrl
is being correctly passed to the container at startup:and when the server is ready, it reports back that it's using
/lab
as the default -- but at the end of this snippet i'm redirected to/tree
instead:i believe this is happening as a consequence of this PR: https://github.com/jupyter/notebook/pull/6941/files
Reproduce
Expected behavior
respect
defaultUrl
when set in the jupyterhub deployment config.Context
notebook 7.0.0rc1 or 7.0.0rc2, JL 4+, JH4+
here's our image config: https://github.com/berkeley-dsep-infra/datahub/blob/staging/deployments/data100-jl4/image/environment.yml
The text was updated successfully, but these errors were encountered: