search

intercity / chef-repo

Set up your server to host Ruby on Rails apps. - Follow us on Twitter: @intercityup
MIT License
417 stars 84 forks source link

Set min_instances for passenger based installations #169

Closed jvanbaarsen closed 9 years ago

jvanbaarsen commented 9 years ago

set passenger_min_instances to 1 so passenger makes sure that one process is always present.

ghost commented 9 years ago

@jvanbaarsen according to the passenger's docs, the default is already 1:

https://www.phusionpassenger.com/documentation/Users%20guide%20Nginx.html#PassengerMinInstances

jvanbaarsen commented 9 years ago

"This option may only occur once, in the http configuration block. The default value is 0."

On Wed, Feb 4, 2015 at 8:20 AM, Oleksandr Iuzikov notifications@github.com wrote:

@jvanbaarsen according to the passenger's docs, the default already 1:

https://www.phusionpassenger.com/documentation/Users%20guide%20Nginx.html#PassengerMinInstances

Reply to this email directly or view it on GitHub: https://github.com/intercity/chef-repo/issues/169#issuecomment-72802201

ghost commented 9 years ago

Just to clarify:

selection_089

jvanbaarsen commented 9 years ago

Hm Now I'm confused :p

On Wed, Feb 4, 2015 at 8:27 AM, Oleksandr Iuzikov notifications@github.com wrote:

Just to clarify:

selection_089

Reply to this email directly or view it on GitHub: https://github.com/intercity/chef-repo/issues/169#issuecomment-72802959

ghost commented 9 years ago

What are you confused with? Is this issue still valid?

jvanbaarsen commented 9 years ago

I have to check.

On Wed, Feb 4, 2015 at 8:50 AM, Oleksandr Iuzikov notifications@github.com wrote:

What are you confused with? Is this issue still valid?

Reply to this email directly or view it on GitHub: https://github.com/intercity/chef-repo/issues/169#issuecomment-72804900

michiels commented 9 years ago

Was this a bug report? Maybe the passenger got killed because of out of memory or a re-deploy?

jvanbaarsen commented 9 years ago

No it was something mentioned by one of the people I talked about for IC. But seems like the default is 1, I've misread it I guess. So for now this issue is invalid. People can change this once we start supporting custom NGINX configs.