-
Notifications
You must be signed in to change notification settings - Fork 133
Failed to start Jupyter service with configurable-http-proxy 4.4.0 #321
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
Comments
Thank you for opening your first issue in this project! Engagement like this is essential for open source projects! 🤗 |
Thanks for reporting this bug, syntax error it sais - i must say i love syntax error bugs compared to all other bugs!!! ❤️ |
I have the same issue! |
@Icare2000 and @MvdB202, I came back from version |
Hi! If you run |
If I run it directly I see the output I put in the top of my issue under bug description, I just started a new VM, manually updated 4.3.2 to 4.4.0, restarted my jupyterhub.service and get the same issue. My Jupyter webconsole turns from being accessible to a My workflow: with ansible I spin up a VM with Ubuntu 18.04, nginx 1.14.0, python 3.6.9, then I install nodejs, jupyterlab 3.0.11, jupyterhub 1.3.0, virtualenv 20.4.3, jupyterlab-git 0.30.0b3 and configurable-http-proxy (used to not specify version, but playbook now specifies 4.3.2). My nginx config for jupyterhub (
My
My
Then I upgrade nbdime and start/enable jupyterhub.service. |
@MvdB202 what version of node is installed ( It could be relevant to know the version of the prom-client library used as well, but I don't know how to get that information. @dtaniwaki do you have a guess on how to resolve this? |
Oh yes sorry, node version v8.17.0. I did an npm -g list with a grep on prom-client but no result. |
@MvdB202 ah! That is likely an issue! Can you see if upgrading to node 12 or later resolves the issue? configurable-http-proxy seem to be failing to require a certain node version, we only test against 10+ and as node 10 is no longer maintained and I think we should enforce node 12 and higher at this point in time. I've seen node 10 tests fail intermittently in our tests as well, and I know old versions of node has caused trouble in the past for us. configurable-http-proxy/.github/workflows/test.yml Lines 65 to 69 in 790a8c7
|
I also tested on my system. Node was outdated. Now it's working with 14! |
Sorry for the late response. I agree that we should follow the maintained node versions and test against them in the CI! |
@consideRatio You were right! I did not think to check the node version before, or thought that was the issue, but I just tested this and it works! Thanks for your help, I will close the topic. |
Bug description
With configurable-http-proxy version 4.4.0, JupyterHub failed to start proxy. Output:
Expected behaviour
I expected JupyterHub service to start as usual with the config proxy adding default route for Hub > http://127.0.0.1:8081
Actual behaviour
May 31 10:37:46 my-instance jupyterhub[14230]: [I 2021-05-31 10:37:46.563 JupyterHub proxy:666] Starting proxy @ http://:8000
How to reproduce
My instance is created with an ansible playbook where I had the npm module install configurable-http-proxy globally. Which now uses version 4.4.0 by default, which triggered this bug.
Currently I have version: '4.3.2' added to my playbook and everything is back up and running.
Your personal set up
jupyterhub version 1.3.0
Python 3.6.9
Ubuntu 18.04
nginx 1.14.0
Configuration
Logs
The text was updated successfully, but these errors were encountered: