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

Allow shutdown when no engines are registered #2769

Merged
merged 2 commits into from Jan 10, 2013

Conversation

dhirschfeld
Copy link

When no engines are registered the Client.shutdown method raises a NoEnginesRegistered exception.
This occurs before the hub shutdown thus preventing the hub from being shutdown.

@dhirschfeld
Copy link
Author

The following code demonstrates the problem:

from IPython.utils.path import locate_profile
from IPython.parallel.apps.launcher import LocalControllerLauncher
from IPython.parallel import Client
controller = LocalControllerLauncher(profile_dir=locate_profile())
controller.start()
try:
    rc = Client()
    rc.shutdown()
    rc.shutdown(hub=True)
finally:
    controller.stop()
#

@minrk
Copy link
Member

minrk commented Jan 10, 2013

Looks clear and right to me.

minrk added a commit that referenced this pull request Jan 10, 2013
Allow shutdown when no engines are registered
@minrk minrk merged commit 821187b into ipython:master Jan 10, 2013
@dhirschfeld dhirschfeld deleted the clean-shutdown2 branch January 10, 2013 20:47
minrk added a commit that referenced this pull request Mar 5, 2013
When no engines are registered the `Client.shutdown` method raises a `NoEnginesRegistered` exception.
This occurs before the hub shutdown thus preventing the hub from being shutdown.
mattvonrocketstein pushed a commit to mattvonrocketstein/ipython that referenced this pull request Nov 3, 2014
Allow shutdown when no engines are registered
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.

None yet

2 participants