Unable to set `pm.status_path`

[doenietzomoeilijk]

Title says it all, really; the current template and vars don't allow you to set pm.status_path for a pool, unless I'm missing something obvious — which is far from impossible.

Is this an actual decision or just something that simply never came up and that you're willing to receive a PR for?

[geerlingguy]

I think it might be simpler (and more flexible) to add in a variable like php_fpm_extra_settings, and then you could define additional settings (any from https://www.php.net/manual/en/install.fpm.configuration.php that aren't already covered).

Then you could define in your playbook:

php_fpm_extra_settings: |
  pm.status_path = /my/path/here

To achieve that, in the existing template, we'd just need to add a line at the bottom:

{{ php_fpm_extra_settings }}

And in defaults/main.yml, a new setting like:

php_fpm_extra_settings: ''

(And a line documenting the new setting in README.)

[shakalandy]

this would be super helpful, indeed.

But what about folks using multiple pools via php_fpm_pools? Would this extra setting be added to all of the pools? Same values make sense if only defined "per pool", or?

[geerlingguy]

@shakalandy - I think (at least from my experience, with smaller installations / single sites), we're starting to get into more narrow use cases, and at that point I'd rather someone forks the role and tailors it to their use case better.

[steinbrueckri]

@geerlingguy @shakalandy I have opened a PR for this topic.

[marcin-github]

It would also fix a couple of other issues (e.g. about clear_env ) and it would fix my issue - because I can't set own environment variables in pool.

[stale[bot]]

This issue has been marked 'stale' due to lack of recent activity. If there is no further activity, the issue will be closed in another 30 days. Thank you for your contribution!

Please read this blog post to see the reasons why I mark issues as stale.

[stale[bot]]

This issue has been closed due to inactivity. If you feel this is in error, please reopen the issue or file a new issue with the relevant details.

[marcin-github]

not stale