sous-chefs / php

Development repository for the php cookbook
https://supermarket.chef.io/cookbooks/php
Apache License 2.0
443 stars 491 forks source link

docs: describe caveat and workaround for multiple fpm pools #366

Closed igolman closed 2 weeks ago

igolman commented 2 weeks ago

Description

Added documentation on handling multiple php-fpm service restarts when provisioning multiple FPM pools. This serves as an addendum to PR 365 and a RFC at once.

While the change in PR 365 was not entirely incorrect (default :delayed notification :timer was explicitly set), it did not address the issue at all.

After investigating the problem further and testing a working solution, I’d like to share my findings.

Since chef-client v18,unified_mode true is the default setting for custom resources.

With unified_mode true enabled:

This means that my previous assertion in PR 365 was incorrect. Although the service resource notification was indeed set to :delayed, due to unified_mode true it was triggered at the end of the action :install block within the fpm_pool resource, not at the end of the chef-client run.

Documentation has been added; feedback is welcome.

Issues Resolved

Caveat and possible Workaround for managing multiple FPM pools documented.

Check List

kitchen-porter commented 2 weeks ago

Released as: 10.2.2