From 51984f06de55453022bc8a048514ace191fb959b Mon Sep 17 00:00:00 2001
From: Vitali Quiering <vitali@quiering.com>
Date: Thu, 16 Feb 2017 10:42:15 +0100
Subject: [PATCH 1/2] Make php.ng.maps.jinja work for php5 on Debian. Fixed
 config dir.

---
 php/ng/map.jinja | 17 +++++++++++------
 1 file changed, 11 insertions(+), 6 deletions(-)

diff --git a/php/ng/map.jinja b/php/ng/map.jinja
index a1c6f7e..ba51d96 100644
--- a/php/ng/map.jinja
+++ b/php/ng/map.jinja
@@ -1400,6 +1400,11 @@
     {%- endif %}
 {%- elif salt['grains.get']('os') == "Debian" %}
     {%- set phpng_version = salt['pillar.get']('php:ng:version', '7.0')|string %}
+    {%- if phpng_version|string == '5' %}
+      {%- set confdir = '/etc/php5' %}
+    {%- else %}
+      {%- set confdir = '/etc/php/' + phpng_version %}
+    {%- endif %}
     {%- set php = salt['pillar.get']('php:ng', {
     'lookup': salt['grains.filter_by']({
                     'Debian': {
@@ -1468,16 +1473,16 @@
                             'zip': 'php' + phpng_version + '-zip',
                         },
                         'fpm': {
-                            'conf': '/etc/php/' + phpng_version + '/fpm/php-fpm.conf',
-                            'ini': '/etc/php/' + phpng_version + '/fpm/php.ini',
-                            'pools': '/etc/php/' + phpng_version + '/fpm/pool.d',
+                            'conf': confdir + '/fpm/php-fpm.conf',
+                            'ini': confdir + '/fpm/php.ini',
+                            'pools': confdir +  '/fpm/pool.d',
                             'service': 'php' + phpng_version + '-fpm',
                             'defaults': odict([
                                 ('global', odict([
                                     ('pid', '/var/run/php' + phpng_version + '-fpm.pid'),
                                     ('error_log', '/var/log/php' + phpng_version + '-fpm.log'),
                                 ])),
-                                ('include', '/etc/php/' + phpng_version + '/fpm/pool.d/*.conf'),
+                                ('include', confdir + '/fpm/pool.d/*.conf'),
                             ]),
                         },
                         'hhvm': {
@@ -1505,10 +1510,10 @@
                             ]),
                         },
                         'cli': {
-                            'ini': '/etc/php/' + phpng_version + '/cli/php.ini',
+                            'ini': confdir + '/cli/php.ini',
                         },
                         'apache2': {
-                            'ini': '/etc/php/' + phpng_version + '/apache2/php.ini',
+                            'ini': confdir + '/apache2/php.ini',
                         },
                     },
                 }),

From 75bbecaf28967e3c4ad5be203bf923226121d832 Mon Sep 17 00:00:00 2001
From: Mario Fritschen <mario.fritschen@surfmedia.de>
Date: Tue, 17 Oct 2017 10:49:42 +0200
Subject: [PATCH 2/2] Add possibility to define default values for php-fpm
 pools

---
 php/ng/fpm/pools_config.sls | 6 ++++++
 pillar.example              | 6 ++++++
 2 files changed, 12 insertions(+)

diff --git a/php/ng/fpm/pools_config.sls b/php/ng/fpm/pools_config.sls
index e3d7aa7..9a66a55 100644
--- a/php/ng/fpm/pools_config.sls
+++ b/php/ng/fpm/pools_config.sls
@@ -10,6 +10,12 @@
 {% set pool_states = [] %}
 
 {% for pool, config in php.fpm.pools.iteritems() %}
+{% if pool == 'defaults' %}{% continue %}{% endif %}
+{% for pkey, pvalues in config.get('settings', {}).iteritems() %}
+{% set pool_defaults = php.fpm.pools.get('defaults', {}).copy() %}
+  {% do pool_defaults.update(pvalues) %}
+  {% do pvalues.update(pool_defaults) %}
+{% endfor %}
 {% set state = 'php_fpm_pool_conf_' ~ loop.index0 %}
 {% set fpath = path_join(config.get('filename', pool), php.lookup.fpm.pools) %}
 
diff --git a/pillar.example b/pillar.example
index c016190..59282d1 100644
--- a/pillar.example
+++ b/pillar.example
@@ -93,6 +93,12 @@ php:
 
       # settings for fpm-pools
       pools:
+        # defaults will apply for each pools settings and can be overwritten by pool settings
+        defaults:
+          user: nginx
+          group: nginx
+          listen: /var/run/php-fpm-default.sock
+
         # name of the pool file to be managed, this will be appended
         # to the path specified in php.ng.lookup.fpm.pools
         'mypool.conf':