-
Notifications
You must be signed in to change notification settings - Fork 38
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
[DH-3] try using a test fork of the chp #5501
[DH-3] try using a test fork of the chp #5501
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks like valid chart yaml to me, based on https://github.com/jupyterhub/zero-to-jupyterhub-k8s/blob/main/jupyterhub/templates/proxy/deployment.yaml and https://z2jh.jupyter.org/en/stable/resources/reference.html#proxy-chp-extracommandlineflags.
At first I thought the timeouts were 1000 days, but I see the units are milliseconds.
hmm, interesting. i did a manual deployment to test this out on a staging hub and got the following error in the chp pod:
|
ok, commenting out
|
ah, it's |
there we go!
|
@consideRatio fyi, this looks quite promising. i've been running the fork without timeouts on our two biggest/most active hubs since friday last week and have only had one here's the memory usage over the past 24 hours on one: i'm keeping a close eye on these two hubs for the rest of the week and will roll out to our remaining hubs on monday. since our core node has plenty of available ram, i'm also bumping the |
@consideRatio pointed me to this... there were a couple of memory leaks in
node-http-proxy
that haven't been rolled in to a release yet, so let's test this out and see if it helps.jupyterhub/configurable-http-proxy#434 (comment)
http-party/node-http-proxy#1559
Jimbly/http-proxy-node16@56283e3
https://github.com/consideRatio/configurable-http-proxy/commits/main/