Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fix reloading php-fpm on trusty & utopic #107

Merged
merged 1 commit into from
Jul 24, 2015
Merged

Fix reloading php-fpm on trusty & utopic #107

merged 1 commit into from
Jul 24, 2015

Conversation

fpletz
Copy link
Contributor

@fpletz fpletz commented Jul 24, 2015

Ubuntu versions newer than precise support reload signals but that feature is disabled by the package maintainer because he builds packages with the same upstart config for precise, too.

But because we actually use reload, refreshing the service doesn't work correctly which leads to php-fpm child processes that never get killed. These processes prevent restarting php-fpm altogether. The only fix is manually killing them.

See https://bugs.launchpad.net/ubuntu/+source/php5/+bug/1242376 for reference.

This patch ensures that reloading is not used on precise and the reload signal USR2 is used for all other supported versions of Ubuntu by creating an upstart override file.

/cc @globin @exi

Ubuntu versions newer than precise support reload signals but that
feature is disabled by the package maintainer because he builds
packages with the same upstart config for precise, too.

But because we actually use reload, refreshing the service doesn't work
correctly which leads to php-fpm child processes that never get killed.
These processes prevent restarting php-fpm altogether. The only fix is
manually killing them.

See https://bugs.launchpad.net/ubuntu/+source/php5/+bug/1242376 for
reference.

This patch ensures that reloading is not used on precise and the reload
signal USR2 is used for all other supported versions of Ubuntu by
creating an upstart override file.

/cc @globin @exi
@fpletz fpletz added bug Something isn't working confirmed labels Jul 24, 2015
exi pushed a commit that referenced this pull request Jul 24, 2015
Fix reloading php-fpm on trusty & utopic
@exi exi merged commit 64a91bd into master Jul 24, 2015
@fpletz fpletz deleted the fix/upstart-reload branch July 24, 2015 15:56
@fpletz fpletz added this to the 3.4.1 milestone Jul 24, 2015
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants