search

jupyterhub / the-littlest-jupyterhub

Simple JupyterHub distribution for 1-100 users on a single server
https://tljh.jupyter.org
BSD 3-Clause "New" or "Revised" License
1.04k stars 339 forks source link

added `remove_named_servers` setting for jupyterhub-idle-culler #881

Closed consideRatio closed 1 year ago

consideRatio commented 1 year ago

This is a duplicate of #855 by @stevejpurves that has been manually rebased to handle conflicts following from rST to MyST. I had to open a new PR to resolve this as I wasn't allowed to push to #855

@stevejpurves is this okay with you to merge? I made one edit besides the rST -> MyST markdown conversions required to solve merge conflicts from #863. You can see that change in a comment below.

Closes #855.

Original PR

If you are using TLJH with named servers, it can be a requirement that the server is removed when culled. This PR is to expose the underlying IdleCuller option, allowing it to be set via tljh-config.

consideRatio commented 1 year ago

Hey @stevejpurves, this is based on your PR in #855, but also manually handling merge conflicts from having merged #863. If its alright with you, I'll go for a merge here!

consideRatio commented 1 year ago

I'm going for a merge, hoping its okay!