capistrano-plugins / capistrano-unicorn-nginx

Capistrano tasks for automatic and sensible unicorn + nginx configuration
MIT License
175 stars 81 forks source link

Use unicorn reload instead of restart #77

Closed redcancode closed 8 years ago

redcancode commented 9 years ago

In case of systemd the restart command does not work the way it should. It is running the restart job of the service unit instead of running the restart function of the System V init script.

My experience on Debian 8.1 and systemd was that every other unit restart did not started up a new master process, only let the old master and workers die.

The solution I choose is running reload instead of restart since reload executes the same function as restart in the System V script and systemd units does not have a reload job so in case of systemd it will fall back to the System V reload call.

mateuszgorniak commented 8 years ago

:+1:

thmsobrmlr commented 8 years ago

:+1: LGTM

ghost commented 8 years ago

:+1:

mateuszgorniak commented 8 years ago

Any problem with this PR?

cweilemann commented 8 years ago

👍 What is holding this up?