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
Welcome back! If you're the original bug submitter, here's where you can edit the bug or add additional notes.
If you forgot your password, you can retrieve your password here.
Password:
Status:
Package:
Bug Type:
Summary:
From: mauro dot stettler at gmail dot com
New email:
PHP Version: OS:

 

 [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)

Pull Requests

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: Sat Dec 21 17:01:58 2024 UTC