php.net |  support |  documentation |  report a bug |  advanced search |  search howto |  statistics |  random bug |  login
Bug #51736 configuration parsing problem
Submitted: 2010-05-04 09:00 UTC Modified: 2010-05-08 10:56 UTC
From: mauro dot stettler at gmail dot com Assigned: fat (profile)
Status: Closed Package: FPM related
PHP Version: Irrelevant OS: suse linux (SLES11)
Private report: No CVE-ID: None
 [2010-05-04 09:00 UTC] mauro dot stettler at gmail dot com
Description:
------------
i tested the revision 298795 of php-fpm and i experienced that the config value of 
min_spare_servers always gets copied into the config value of max_spare_servers, 
which then means that start_servers will also have to be equal to this,

Test script:
---------------
; Sets the number of server processes created on startup.
; Used only with 'dynamic' pm.style
; default : min_spare + (max_spare - min_spare) / 2
;
pm.start_servers = 50

; Sets the desired minimum number of idle server processes.
; Used only with 'dynamic' pm.style
; It's mandatory when pm is set to dynamic
;
pm.min_spare_servers = 10
; Sets the desired maximum number of idle server processes.
; Used only with 'dynamic' pm.style
; It's mandatory when pm is set to dynamic
;
pm.max_spare_servers = 100


Expected result:
----------------
should be able to start php-fpm server

Actual result:
--------------
the result is that on start of php-fpm server, the fpm gives out a config error if 
you use the above pasted configuration

Patches

bug51736.patch (last revision 2010-05-08 08:22 UTC by fat@php.net)

Add a Patch

Pull Requests

Add a Pull Request

History

AllCommentsChangesGit/SVN commitsRelated reports
 [2010-05-04 09:11 UTC] fat@php.net
-Assigned To: +Assigned To: fat
 [2010-05-08 10:22 UTC] fat@php.net
The following patch has been added/updated:

Patch Name: bug51736.patch
Revision:   1273306941
URL:        http://bugs.php.net/patch-display.php?bug=51736&patch=bug51736.patch&revision=1273306941
 [2010-05-08 10:23 UTC] fat@php.net
Can you please try the attached patch to confirm it fixes the bug ?

Thx
++ Jerome
 [2010-05-08 10:52 UTC] fat@php.net
Automatic comment from SVN on behalf of fat
Revision: http://svn.php.net/viewvc/?view=revision&revision=299141
Log: Fix #51736, wrong checks on dynamic parameters
 [2010-05-08 10:54 UTC] fat@php.net
This bug has been corrected on revision 299141.
 [2010-05-08 10:56 UTC] fat@php.net
-Status: Assigned +Status: Closed
 [2010-05-08 10:56 UTC] fat@php.net
fixed in revision 299141
 
PHP Copyright © 2001-2024 The PHP Group
All rights reserved.
Last updated: Fri Mar 29 04:01:29 2024 UTC