Skip to content

Commit

Permalink
Merge pull request #32 from syphernl/fix/zendserver_multi_php
Browse files Browse the repository at this point in the history
Simplify php-fpm config extension for ZS
  • Loading branch information
syphernl committed Sep 19, 2016
2 parents 98d362f + cc6d693 commit f8258b4
Showing 1 changed file with 2 additions and 18 deletions.
20 changes: 2 additions & 18 deletions vhosting/webstack/zendserver.sls
Original file line number Diff line number Diff line change
Expand Up @@ -15,30 +15,14 @@ include:

{%- if webserver == 'nginx' and not disable_webserver %}
## Re-configure PHP-FPM to allow multiple pools to be used
{% if salt['pillar.get']('phpfpm:php_versions', [])|length > 0 %}
php5-fpm:
service.running:
- enable: True
- reload: True
- require:
- file: /usr/local/zend/etc/fpm.d
- file: /usr/local/zend/etc/php-fpm.conf
{%- else %}
extend:
php5-fpm:
service.running:
- enable: True
- reload: True
- require:
- file: /usr/local/zend/etc/php-fpm.conf
{%- endif %}

/usr/local/zend/etc/php-fpm.conf:
file.uncomment:
- regex: ^include=etc
- char: ;
- require:
- pkg: zendserver
- require_in:
- service: php5-fpm
- watch_in:
- service: zendserver
{%- endif %}
Expand Down

0 comments on commit f8258b4

Please sign in to comment.