ansible-role-php icon indicating copy to clipboard operation
ansible-role-php copied to clipboard
verified publisher icon geerlingguy

feat: allow the configuration of any opcache variable

Open jpiron opened this issue 1 year ago • 4 comments

This removes the need to update the role to support new opcache variables.

jpiron avatar May 15 '24 16:05 jpiron

Cool, this would make my PR #416 obsolete, but keep the default, right?

shakalandy avatar May 15 '24 19:05 shakalandy

Yes, except for the php_realpath_cache_ttl variable.
And indeed it keeps all the defaults as long as you don't define the php_opcache_* variables.
I updated the README to make it more clear.

jpiron avatar May 16 '24 05:05 jpiron

@geerlingguy could we get this one merged? I would adapt #416 to include php_realpath_cache_ttl only - thanks!

shakalandy avatar May 16 '24 15:05 shakalandy

This pr 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.

github-actions[bot] avatar Sep 14 '24 10:09 github-actions[bot]

This pr 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.

github-actions[bot] avatar Jan 25 '25 10:01 github-actions[bot]

This pr 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.

github-actions[bot] avatar Mar 29 '25 10:03 github-actions[bot]