-
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
Allow redis compression options to be specified during setup:install
process
#21053
Allow redis compression options to be specified during setup:install
process
#21053
Conversation
…ession-library for cache and pagecache
…ing up test count.
Hi @cmacdonald-au. Thank you for your contribution
For more details, please, review the Magento Contributor Assistant documentation |
Hi @aleron75, thank you for the review. |
@cmacdonald-au thank you for contributing. Please accept Community Contributors team invitation here to gain extended permissions for this repository. |
✔️ QA passed |
Hi @cmacdonald-au, thank you for your contribution! |
…setup:install` process #21053
Description (*)
When installing magento, "some" redis cache specific options can be specified. The defaults are usually okay, but in some scenarios they need to be changed.
This PR provides the ability to set the compress_data and compression_lib options consistently for default/backend/redis and page_cache/redis.
Providing these options is preferred over hand-rolled
app/etc/env.php
"fiddling" when executing a repeatable installation test for the purpose of environment validation.Fixed Issues (if relevant)
Manual testing scenarios (*)
--cache-backend=redis --cache-backend-redis-server=127.0.0.1 --cache-backend-redis-port=6379 --cache-backend-redis-db=3 --page-cache=redis --page-cache-redis-server=127.0.0.1 --page-cache-redis-port=6379 --page-cache-redis-db=4 --cache-backend-redis-compression-lib=lzf --cache-backend-redis-compress-data=1
app/etc/env.php
Cm_Cache_Backend_Redis
will throw an exception if an invalidcompression_lib
(eg:notavalidlib
) is supplied andcompress_data
is enabled.compress_data
is disabled (compression will not be attempted).Contribution checklist (*)