search

yevgenko / cookbook-php-fpm

Installs/Configures php-fpm
http://community.opscode.com/cookbooks/php-fpm
Apache License 2.0
68 stars 117 forks source link

Disable default www pool if not defined in pools attribute. #53

Closed razor-x closed 10 years ago

razor-x commented 10 years ago

The package that's installed (at least on Ubuntu) will add /etc/php5/fpm/pool.d/www.conf. Thus, even if this pool is not in node['php-fpm']['pools'], it will still be enabled after the node converges.

This removes this stock pool unless it is detected in node['php-fpm']['pools'].

razor-x commented 10 years ago

I removed the conditional for the default pool as per your comment.

andytson commented 10 years ago

I'm not keen on this removing the pool every converge, it'd probably have been better to keep the conditional