As of version 6, redis has native TLS support.
This merge request adds support for the most basic parameters of this feature.
After multi instance was added we should let redis-sentinel watch over these instances
This will close following Issues:
Thank you for contributing to this project!
Replace this comment with a description of your pull request.
Replace this comment with the list of issues or n/a.
This commit introduces the latest config for Redis6 and Sentinel, with new parameters. Additionally, the MemoryPolicy type is introduced with some small adjustments in the rest of the code.
As discussed previously on voxpupuli IRC this is a base commit, and depending on the review I will make changes accordingly.
This PR introduces the possibility to install
redis::sentinel standalone, without
redis::server. It comes in handy when you need your sentinels to be for example on a different node than your redis server.
The change itself is a pretty straightforward, I am just adding a
require_redis parameter, that makes the requirement of
redis class conditional. The default value is true, which includes the
redis and it is therefore backward compatible with the current setup.
The only downside of this approach I see currently is that in case someone needs a standalone
redis-sentinel from a managed repository, the repository must be added manually as the
redis::preinstall is no longer included (as was previously with the
Thank you for any input :)