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

Exception in reaper.rb:38 #6

Closed
bitboxer opened this issue Jan 9, 2013 · 6 comments
Closed

Exception in reaper.rb:38 #6

bitboxer opened this issue Jan 9, 2013 · 6 comments

Comments

@bitboxer
Copy link
Contributor

bitboxer commented Jan 9, 2013

I am getting this error:

ruby-1.9.2-p320/gems/zombie_passenger_killer-0.2.5/lib/zombie_passenger_killer/reaper.rb:38:in `hunt_zombies': private method `select' called for nil:NilClass (NoMethodError)

can you check that?

@grosser
Copy link
Owner

grosser commented Jan 9, 2013

looks like it could not find any processes in passenger status because passenger-status had a non-zero exit status,
a pull request with improved reporting e.g. via warn would be great :)

@bitboxer
Copy link
Contributor Author

bitboxer commented Jan 9, 2013

Hm, i am not an expert in passenger-status, but if this is an expected behavior of passenger, I would change the line to something like this:

high_load = if !active_processes_in_processlist.nil? && @max_high_cpu 

Is that correct?

@grosser
Copy link
Owner

grosser commented Jan 9, 2013

I think it's more of an configuration issue, passenger-status exists with
error because it cannot be executed or something like that, hiding that
would make it harder to debug -> warn users with a descriptive error message

try to gem open the gem and add a few puts to see what exectly is going on
-> reproduce on the console

On Wed, Jan 9, 2013 at 2:04 PM, Bodo Tasche notifications@github.comwrote:

Hm, i am not an expert in passenger-status, but if this is an expected
behavior of passenger, I would change the line to something like this:

high_load = if !active_processes_in_processlist.nil? && @max_high_cpu

Is that correct?


Reply to this email directly or view it on GitHubhttps://github.com//issues/6#issuecomment-12069310.

@bitboxer
Copy link
Contributor Author

Okay, I got the issue. It was a bundler problem. Maybe I should send you a pull request that outputs the passenger-status output if the result is not 0 ?

@grosser
Copy link
Owner

grosser commented Jan 11, 2013

That would be great, it could even just raise, everything is better then
this cryptic error :)

On Fri, Jan 11, 2013 at 12:11 AM, Bodo Tasche notifications@github.comwrote:

Okay, I god the issue. It was a bundler Problem. Maybe I should send you a
pull request that outputs the passenger-status output if the result is not
0 ?


Reply to this email directly or view it on GitHubhttps://github.com//issues/6#issuecomment-12135107.

@bitboxer
Copy link
Contributor Author

I just send you a pull request for this

grosser added a commit that referenced this issue Jan 14, 2013
raise exception if passenger returns not 0 , #6
@grosser grosser closed this as completed Jan 14, 2013
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

2 participants