Req #67553 [NEW]: Add SIGHUP as a reload signal

From: Date: Wed, 02 Jul 2014 10:12:34 +0000
Subject: Req #67553 [NEW]: Add SIGHUP as a reload signal
Groups: php.bugs 
Request: Send a blank email to php-bugs+get-186419@lists.php.net to get a copy of this message 
From:             ondrej
Operating system: Linux
PHP version:      5.4.30
Package:          FPM related
Bug Type:         Feature/Change Request
Bug description:Add SIGHUP as a reload signal

Description:
------------
Both upstart and systemd are using SIGHUP as a default signal for reload
action and the init scripts needs special configuration because FPM uses
SIGUSR2 for reload.

Since SIGHUP is unused in FPM there's no harm in adding it as another
reload signal.

Patch attached.

Applies to all supported versions of PHP.

Test script:
---------------
service php5-fpm reload

Expected result:
----------------
PHP FPM reloaded

Actual result:
--------------
PHP FPM in invalid state - main process crashed without killing children
processes.

-- 
Edit bug report at https://bugs.php.net/bug.php?id=67553&edit=1
--

Thread (3 messages)

« previous php.bugs (#186419) next »