ronnix/fabtools — pull request #26 by danfairs (2012-09-06 09:58:18+00:00)
Title: Use 'update' rather than 'reload' to avoid unnecessary restarting

Hi,

Just wanted to get your opinion on this change. It adds an 'update' action to supervisor.py, and uses that instead of reload when supervisor service conf files change.

Reload's quite heavy-handed, and causes supervisor itself to restart, and all managed processes. I found occasionally that child processes didn't shut down properly - especially when I add several services in quick succession, as my deploys often do - and it isn't necessary to restart all services just because one has changed.

What do you think?

Cheers,
Dan