Conversation
|
Thank you for your contribution. This pull request has been marked as stale as it has not passed the automated tests and there was no activity for the last 14 days. This pull request will be closed due to inactivity after 60 days, if no action is taken. |
|
At dev, I would consider creating a dedicated valkey plug-in:
|
|
Thank you for your contribution. This pull request has been marked as stale as it has not passed the automated tests and there was no activity for the last 14 days. This pull request will be closed due to inactivity after 60 days, if no action is taken. |
e75dba3 to
dcc48e8
Compare
General information
Because of a former license change some linux distributions migrate from redis to its fork valkey (e.g. SLES 15 SP7 does not contain redis anymore. It was replaced with valkey). Same as the MySQL plugin also supports its fork MariaDB the redis plugin should support Valkey.
mk_redis already works with Valkey if configured explicitly and either redis-cli is present or is a symlink to valkey-cli.
Proposed changes
Use the mk_redis plugin to detect valkey-server instances and auto-detect the available cli.
This requires two modifications:
As a bonus this PR moves the special case for the redis-cli of a CheckMK site also into the redis_args function. So configuration of REDIS_CLI_COMMAND is all in one place.