Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Raising value of php.memory_consumption from 128 to 256 at Dockerfile level for NC > versions 30 #2352

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

neokyx
Copy link

@neokyx neokyx commented Dec 17, 2024

Hello,

According to the issue reported here
I'm humbly suggesting this change to solve the (too many) recurrent log warning and as requested on :

PHP OPcache module is not configured correctly The OPcache buffer is almost full. To make sure that all scripts can be retained in cache, it is recommended to add "opcache.memory_consumption" to your PHP configuration with a value higher than "128"... For more details check the documentation.

It indeed appears frequently, even on modest Nextcloud Installation (4/5 apps on a 2vcpus and 8Go Ram for server configuration)

The PHP OPcache module is not properly configured. The OPcache buffer is nearly full. To assure that all scripts can be hold in cache, it is recommended to apply "opcache.memory_consumption" to your PHP configuration with a value higher than "128"

Signed-off-by: Spyderfoo <[email protected]>
Suggesting (#line 101) to improve opcache.memory_consumption to value 256 instead of 128, in order to avoid the following recurrent warning, even when a rather light Nextcloud installation, with very few (4/5) apps is used.
```
"The PHP OPcache module is not properly configured. The OPcache buffer is nearly full. To assure that all scripts can be hold in cache, it is recommended to apply "opcache.memory_consumption" to your PHP configuration with a value higher than "128".
```

Signed-off-by: Spyderfoo <[email protected]>
Signed-off-by: Spyderfoo <[email protected]>
@joshtrichards
Copy link
Member

Hi @neokyx! You may wish to join one of the existing PRs/discussions on this topic. #2090 comes to mind.

@neokyx
Copy link
Author

neokyx commented Dec 17, 2024

Hi @joshtrichards ! Sorry, I should have looked for an open topic before submitting my PR...😅 Thank you for the answer !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants