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

Cull idle kernels #80

Closed
wants to merge 1 commit into from
Closed

Conversation

jtpio
Copy link
Member

@jtpio jtpio commented Mar 13, 2019

Expose cull_idle_timeout and cull_interval to be able to cull idle kernels when using voila as a standalone app. cull_busy and cull_connected shouldn't be needed (for now) since there is one kernel started at every page load.

This is especially useful for long running voila instances to be able to free resources.

Usage: python -m voila --Voila.cull_idle_timeout=10 --Voila.cull_interval=10

Potential fix for #55

@maartenbreddels
Copy link
Member

I didn't know this existed, by I think it's a useful thing to configure for voila.

I'm working on a PR to use the Configurable machinery of traitlets to help out instead.

@jtpio
Copy link
Member Author

jtpio commented Mar 14, 2019

#84 looks really good and is more generic.

Let's close this PR then.

@jtpio jtpio closed this Mar 14, 2019
@jtpio jtpio deleted the cull-kernels branch March 15, 2019 16:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants