thias / puppet-php

Puppet module to manage PHP
Other
49 stars 67 forks source link

Added process.priority #25

Closed Nyholm closed 10 years ago

Nyholm commented 10 years ago

You may now set a priority on a pool. With this feature you may have one low priority pool for background processes and a other pool with normal priority for your webpage visitors.

Here is a link to that feature request: https://bugs.php.net/bug.php?id=62160

This was added in PHP 5.4.4 if I'm not mistaken.

thias commented 10 years ago

This has been added in a1fd9fe685f4a752b4ec611e63c4ccca3c8968d1 along with the per-pool setting and a few others, after looking at https://raw.githubusercontent.com/php/php-src/master/sapi/fpm/php-fpm.conf.in

Nyholm commented 10 years ago

I'm afraid you made a misstake when copying my changes. Process_priority is a property of the pool config, not the daemon.

Have you considered using Github's merge function?

Nyholm commented 10 years ago

Bump @thias