After multi instance was added we should let redis-sentinel watch over these instances
This will close following Issues:
Close #319
Close #223
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 redis
class)
Thank you for any input :)
Avoid using this copy with destination and source beeing equal
Remark: generally it's a bad idea to use an exec statement to copy a file. Since this way we cannot see diffs etc.
A better way would be to use a file{ $destination: source=> $source} statement, but running with noop makes it fail then !
Signed-off-by: Martin KolacĂ martin.kolaci@lmc.eu
<!--
Thank you for contributing to this project!
-->
Added replicaof parameter in config
Fixes #401
modulesync 5.4.0
<!--
Thank you for contributing to this project!
-->
Avoid systemd-related code when running on non-Linux operating systems.
No issue found, just noticed that FreeBSD support is broken.
<!--
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.
Use format:
Fixes #123
Fixes #124
-->
minor change to the redis.conf
EPP template to make it possible to disable rdb_save_incremental_fsync
Fixes #494
<!--
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.
Use format:
Fixes #123
Fixes #124
-->
Process the configuration file redis.conf
template twice if some of the templates values are deferred.
Currently with deferred values the resulting deferred template cannot be processed since it contains
complex datatypes from stdlib in particular.
This is a redis specific solution that may arrive generally one day in https://github.com/puppetlabs/puppetlabs-stdlib/pull/1425
Fixes #513