Skip to content

Conversation

@mayrstefan
Copy link
Contributor

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:

  1. detect running instances with binaries named valkey-server
  2. make use of the valkey-cli if available

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.

@github-actions
Copy link

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.
Please take a look at the ‘Checks’ section for details on the test results and make the necessary changes.

This pull request will be closed due to inactivity after 60 days, if no action is taken.

@github-actions github-actions bot added the Stale Scheduled for auto-close label Oct 27, 2025
@martinhv
Copy link
Member

At dev, I would consider creating a dedicated valkey plug-in:

  • Eventually users will forget the relation valkey = fork of redis. So they will be looking for valkey, not redis
  • Changes in valkey will impact the redis agent plug-in visa versa (example: 7ed5059)
  • Makes automated testing more difficult

@github-actions github-actions bot removed the Stale Scheduled for auto-close label Oct 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants