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

Web process doesn't terminate with Thin 1.2.11 #10

Closed
adamwiggins opened this issue Apr 9, 2011 · 4 comments
Closed

Web process doesn't terminate with Thin 1.2.11 #10

adamwiggins opened this issue Apr 9, 2011 · 4 comments

Comments

@adamwiggins
Copy link

Breaking out of Foreman doesn't kill the web process with the latest version of Thin:

$ foreman start
12:18:36 web.1     | started with pid 55657
12:18:38 web.1     | == Sinatra/1.2.1 has taken the stage on 5000 for development with backup from Thin
12:18:38 web.1     | >> Thin web server (v1.2.11 codename Bat-Shit Crazy)
12:18:38 web.1     | >> Maximum connections set to 1024
12:18:38 web.1     | >> Listening on 0.0.0.0:5000, CTRL+C to stop
^C12:18:39 web.1     | process exiting
SIGINT received
12:18:39 system    | terminating
12:18:39 system    | killing web in pid 55657
12:18:39 web.1     | process terminated
12:18:39 system    | terminating
$ ps aux | grep ruby
adam     55659   0.1  0.3  2470300  26208   ??  S    12:18PM   0:01.63 ruby web.rb -p 5000

Switching back to Thin 1.2.7 removes this problem, so I'm guessing that the newer version forks differently.

Here's a sample app: https://gist.github.com/911691

@rodrigotassinari
Copy link

+1, exactly the same issue

@moomerman
Copy link

Yeah, same here.

@ddollar
Copy link
Owner

ddollar commented May 13, 2011

Fixed in 191581f

@ddollar ddollar closed this as completed May 13, 2011
@ddollar
Copy link
Owner

ddollar commented May 13, 2011

Was released in foreman 0.15.0

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

No branches or pull requests

4 participants