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

limit the number of simultaneous servers #46

Open
rokroskar opened this issue Sep 12, 2018 · 5 comments
Open

limit the number of simultaneous servers #46

rokroskar opened this issue Sep 12, 2018 · 5 comments
Labels
enhancement New feature or request help wanted Extra attention is needed

Comments

@rokroskar
Copy link
Member

rokroskar commented Sep 12, 2018

When a user requests a new server, check how many are already running and if that number is equal to MAX_USER_SERVERS then shut down the oldest one before starting the new one.

Note (@leafty): shutdown oldest from the same (project, user). The idea is that the user is iterating with his/her Docker image

@rokroskar rokroskar added the enhancement New feature or request label Sep 12, 2018
@rokroskar rokroskar added this to the 0.2.1 milestone Sep 12, 2018
@rokroskar rokroskar added the help wanted Extra attention is needed label Sep 12, 2018
@rokroskar rokroskar removed this from the 0.3.0 milestone Nov 26, 2018
@lorenzo-cavazzi
Copy link
Member

lorenzo-cavazzi commented Mar 3, 2020

this should be possible starting from version 1.0 by using c.JupyterHub.named_server_limit_per_user https://jupyterhub.readthedocs.io/en/stable/changelog.html#new-features

@rokroskar
Copy link
Member Author

We probably want to resolve #222 and SwissDataScienceCenter/renku-ui#686 before imposing these limits.

@olevski
Copy link
Member

olevski commented Feb 1, 2022

Should this be moved to Amalthea?

@rokroskar
Copy link
Member Author

Not sure I think it should be the notebook service concern since it's also the one initiating new sessions. Why would it go to amalthea? It doesn't really know the concept of a user afaik.

@olevski
Copy link
Member

olevski commented Feb 1, 2022

Yeah that is a good point @rokroskar I did not read the description well enough. I thought this talked about limiting the total number of all servers on a cluster. Not the number per user.

@Panaetius Panaetius moved this to Backlog in renku-python May 18, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request help wanted Extra attention is needed
Projects
Status: Backlog
Development

No branches or pull requests

3 participants