mirror of
https://github.com/jupyter/notebook.git
synced 2025-01-12 11:45:38 +08:00
Merge pull request #3665 from mpacer/reply_timeout
add kernel_info_timeout traitlet for slow kernel start/restart
This commit is contained in:
commit
2c061a45ff
@ -104,7 +104,8 @@ class ZMQChannelsHandler(AuthenticatedZMQStreamHandler):
|
||||
|
||||
@property
|
||||
def kernel_info_timeout(self):
|
||||
return self.settings.get('kernel_info_timeout', 10)
|
||||
km_default = self.kernel_manager.kernel_info_timeout
|
||||
return self.settings.get('kernel_info_timeout', km_default)
|
||||
|
||||
@property
|
||||
def iopub_msg_rate_limit(self):
|
||||
|
@ -19,7 +19,7 @@ from tornado.ioloop import IOLoop, PeriodicCallback
|
||||
from jupyter_client.session import Session
|
||||
from jupyter_client.multikernelmanager import MultiKernelManager
|
||||
from traitlets import (Any, Bool, Dict, List, Unicode, TraitError, Integer,
|
||||
Instance, default, validate
|
||||
Float, Instance, default, validate
|
||||
)
|
||||
|
||||
from notebook.utils import to_os_path, exists
|
||||
@ -94,6 +94,18 @@ class MappingKernelManager(MultiKernelManager):
|
||||
"""
|
||||
)
|
||||
|
||||
kernel_info_timeout = Float(60, config=True,
|
||||
help="""Timeout for giving up on a kernel (in seconds).
|
||||
|
||||
On starting and restarting kernels, we check whether the
|
||||
kernel is running and responsive by sending kernel_info_requests.
|
||||
This sets the timeout in seconds for how long the kernel can take
|
||||
before being presumed dead.
|
||||
This affects the MappingKernelManager (which handles kernel restarts)
|
||||
and the ZMQChannelsHandler (which handles the startup).
|
||||
"""
|
||||
)
|
||||
|
||||
_kernel_buffers = Any()
|
||||
@default('_kernel_buffers')
|
||||
def _default_kernel_buffers(self):
|
||||
@ -305,7 +317,7 @@ class MappingKernelManager(MultiKernelManager):
|
||||
kernel.session.send(channel, "kernel_info_request")
|
||||
channel.on_recv(on_reply)
|
||||
loop = IOLoop.current()
|
||||
timeout = loop.add_timeout(loop.time() + 30, on_timeout)
|
||||
timeout = loop.add_timeout(loop.time() + self.kernel_info_timeout, on_timeout)
|
||||
return future
|
||||
|
||||
def notify_connect(self, kernel_id):
|
||||
@ -434,4 +446,3 @@ class MappingKernelManager(MultiKernelManager):
|
||||
self.log.warning("Culling '%s' kernel '%s' (%s) with %d connections due to %s seconds of inactivity.",
|
||||
kernel.execution_state, kernel.kernel_name, kernel_id, connections, idle_duration)
|
||||
self.shutdown_kernel(kernel_id)
|
||||
|
||||
|
Loading…
Reference in New Issue
Block a user