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

Don't wait for container to stop if the client disconnects. #589

Merged
merged 2 commits into from
Jul 16, 2015

Conversation

ejholmes
Copy link
Contributor

This fixes #587. Turns out, a lot of what I was doing previously is unnecessary. AttachToContainer will block while the Read'ing from the connection doesn't return io.EOF, so there's no need to WaitContainer. At that point in time, the container is already stopped because it exited, so there's no need to stop the container and we can just remove it.

If the user killall emp's then the container will still be removed because the connection closes and returns io.EOF.

@phobologic
Copy link
Contributor

👍

ejholmes added a commit that referenced this pull request Jul 16, 2015
Don't wait for container to stop if the client disconnects.
@ejholmes ejholmes merged commit fbcbc45 into master Jul 16, 2015
@ejholmes ejholmes deleted the remove-container branch July 16, 2015 21:00
@ejholmes ejholmes assigned ejholmes and unassigned ejholmes Jun 30, 2016
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.

One off tasks don't get stopped
2 participants