-
Notifications
You must be signed in to change notification settings - Fork 9.3k
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
Redis cache grows unilimmited #25487
Comments
Hi @toxix. Thank you for your report.
Please make sure that the issue is reproducible on the vanilla Magento instance following Steps to reproduce. To deploy vanilla Magento instance on our environment, please, add a comment to the issue:
For more details, please, review the Magento Contributor Assistant documentation. @toxix do you confirm that you were able to reproduce the issue on vanilla Magento instance following steps to reproduce?
|
I experienced a situation on production system with 15 stores and organic trafic, where the redis cache grows to over 30GB. It looks like magento is putting a lot of entries into the redis cache without any expiary. I needed to limit redis memory to prevent the server from dying, not how it should be. This had some performance influences. For this issue there is actualy a fix in place which just need to be pulled in. This commit is do this. With the fix in place the redis stabalized at arround 3GB. Over all performance is was also increased with this change. Referces: [1] colinmollenhour/Cm_Cache_Backend_Redis@bc63e72 [2] magento@32058c7 [3] magento#25487
@toxix Having the same problem but only at certain periods it will grow unlimited. I will see if your fix works too. |
Same is also on Magento version 2.1.7. Did install only that fix, but nothing has really changed. |
Hi @toxix. Thank you for your report.
The fix will be available with the upcoming 2.4.0 release. |
Preconditions (*)
Steps to reproduce (*)
Expected result (*)
Actual result (*)
Further information
The only way to prevent our production system server from dying was to set a hard limit for Redis memory consumption. The relying on the memory limit of redis has some performance impact and there are always things cleared that don't need to and vice versa. This should be handled by the application with setting an resonable ttl.
fix
This is actualy an known bug and already long time fixed in an dependency of magento core composer file. See also colinmollenhour/Cm_Cache_Backend_Redis@bc63e72
But this is not included in current magento version Last update of this core depedency was in September 2018: 32058c7
Applying this change as a hotfix, actualy stabalize our production redis at 3,5GB ram usage, instead of growing over 30GB. And also boost the performance of the system quiete a bit.
Hot Fix
Add the following requirement to your project composer.json file:
Fix
I will File a PR with an updated composer dependency to the magento core and reference this issue.
The text was updated successfully, but these errors were encountered: