Skip to content

Commit

Permalink
Fix kernel shutdown behavior
Browse files Browse the repository at this point in the history
  • Loading branch information
blink1073 committed Apr 29, 2022
1 parent 726948f commit 40abb76
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 9 deletions.
14 changes: 6 additions & 8 deletions jupyter_server/services/kernels/kernelmanager.py
Original file line number Diff line number Diff line change
Expand Up @@ -398,10 +398,9 @@ def shutdown_kernel(self, kernel_id, now=False, restart=False):
if kernel_id in self._pending_kernel_tasks:
task = self._pending_kernel_tasks.pop(kernel_id)
task.cancel()
return

self.stop_watching_activity(kernel_id)
self.stop_buffering(kernel_id)
else:
self.stop_watching_activity(kernel_id)
self.stop_buffering(kernel_id)

self.pinned_superclass.shutdown_kernel(self, kernel_id, now=now, restart=restart)

Expand Down Expand Up @@ -661,10 +660,9 @@ async def shutdown_kernel(self, kernel_id, now=False, restart=False):
if kernel_id in self._pending_kernel_tasks:
task = self._pending_kernel_tasks.pop(kernel_id)
task.cancel()
return

self.stop_watching_activity(kernel_id)
self.stop_buffering(kernel_id)
else:
self.stop_watching_activity(kernel_id)
self.stop_buffering(kernel_id)

# Finish shutting down the kernel before clearing state to avoid a race condition.
return await self.pinned_superclass.shutdown_kernel(
Expand Down
2 changes: 1 addition & 1 deletion tests/services/kernels/test_api.py
Original file line number Diff line number Diff line change
Expand Up @@ -158,7 +158,7 @@ async def test_main_kernel_handler(jp_fetch, jp_base_url, jp_serverapp, pending_


@pytest.mark.timeout(TEST_TIMEOUT)
async def test_kernel_handler(jp_fetch, pending_kernel_is_ready):
async def test_kernel_handler(jp_fetch, jp_serverapp, pending_kernel_is_ready):
# Create a kernel
r = await jp_fetch(
"api", "kernels", method="POST", body=json.dumps({"name": NATIVE_KERNEL_NAME})
Expand Down

0 comments on commit 40abb76

Please sign in to comment.