Skip to content

change cull_idle_timeout_minimum to 1 from 300 #2910

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.

Sign up for GitHub

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

Merged
takluyver merged 3 commits into from
Nov 1, 2017
Merged

change cull_idle_timeout_minimum to 1 from 300 #2910

Changes from 2 commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
241c7e1
change cull_idle_timeout_minimum to 1 from 500
mheilman Oct 9, 2017
f8ef573
added warning about culling kernels with poor network connections
mheilman Oct 11, 2017
b31194d
removed extraneous cull_idle_timeout_minimum
mheilman Oct 20, 2017
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 4 additions & 3 deletions notebook/services/kernels/kernelmanager.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -61,11 +61,12 @@ def _update_root_dir(self, proposal):
raise TraitError("kernel root dir %r is not a directory" % value)
return value

cull_idle_timeout_minimum = 300 # 5 minutes
cull_idle_timeout_minimum = 1
cull_idle_timeout = Integer(0, config=True,
help="""Timeout (in seconds) after which a kernel is considered idle and ready to be culled. Values of 0 or
lower disable culling. The minimum timeout is 300 seconds (5 minutes). Positive values less than the minimum value
will be set to the minimum."""
lower disable culling. The minimum timeout is 1 second. Positive values less than the minimum value
will be set to the minimum. Very short timeouts may result in kernels being culled for users with poor
Copy link
Member

@takluyver takluyver Oct 20, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nitpick: as this is an integer option, there are no longer meaningful positive values less than the minimum. I'd drop that sentence.

network connections."""
)

cull_interval_default = 300 # 5 minutes
Expand Down