-
Notifications
You must be signed in to change notification settings - Fork 80
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
"rerun foreman start", the first re-run fails with "Errno::EADDRINUSE" (Ubuntu 12.10) #41
Comments
I'm experiencing the same issue - any ideas on how to fix that ? I have tried multiple options for rerun (such as --signal KILL) but could not find any solution for it. |
Sorry, I haven't worked on this issue yet... What version of foreman are I would think On Wed, Jan 1, 2014 at 3:52 AM, Alan Rubin [email protected] wrote:
Alex Chaffee - [email protected] |
Same issue here. My foreman version is 0.63.0 Hope it helps!, thanks |
Here's the relevant code: https://github.com/alexch/rerun/blob/master/lib/rerun/runner.rb#L218-L241 It looks like foreman is responding to SIGINT by sending SIGKILL to all its children. But it takes longer than 2 seconds for the children to die (how morbid!) so we escalate to sending SIGKILL and restart forecefully. Yeah, maybe increasing the SIGINT timeout would do it. I'll bump up both timeouts to 5 seconds and push to master; you can then clone it locally and run A better solution would be to expand the
which would do the current logic. |
Stable Mac gems for rerun (0.10.0) and foreman (0.60.0) appear to still manifest this issue. Should those gems be more recent? |
Seeing this on ubuntu 7 years later :( |
This is probably happening due to ddollar/foreman#779 in foreman, that's address by this PR: ddollar/foreman#780 – that PR has been released in an foreman fork called |
ha, same problem on overmind :[ |
"foreman" recieve SIGKILL, so "rackup" not killed by "foreman" (i can see it in a process list on port 5000), so "foreman" can't start second time.
Maybe time intervals between signals should be in options?
The text was updated successfully, but these errors were encountered: