Added sentinel exporter support - #8
Merged
Nick-Andreano merged 1 commit intoAug 7, 2026
Merged
Conversation
Nick-Andreano
requested review from
Amunagala-itential,
Madias2222,
kvelarde-itential and
steven-schattenberg-itential
August 7, 2026 15:38
steven-schattenberg-itential
approved these changes
Aug 7, 2026
kvelarde-itential
approved these changes
Aug 7, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
prometheus_exporters.yml's "Install Redis exporter" play already targeted redis_sentinel hosts via its hosts: redis* pattern, but pointed the exporter at the default Redis port (6379) — which doesn't exist on a Sentinel-only host, since Sentinel listens on 26379. The exporter would fail to connect there instead of exposing Sentinel metrics.
This sets redis_exporter_addr to redis://localhost:26379 when the host is in the redis_sentinel group (otherwise unchanged at 6379). The existing oliver006/redis_exporter binary detects Sentinel mode automatically and exposes redis_sentinel_* metrics once pointed at the right port — no new exporter binary or role needed. No changes were required to scrape_configs.j2; its ^redis_.* group loop already includes redis_sentinel hosts in the redis_exporter scrape job.
Also fixed a stale reference in the README to redis_prometheus_user_enabled, renamed to redis_monitor_user_enabled in deployer v4.
Tested against a live 3-node EC2 lab (1 redis_master, 1 redis_sentinel, 1 prometheus) — confirmed Prometheus successfully scrapes redis_sentinel_masters, redis_sentinel_master_status, etc. from the sentinel host, while the master host's exporter continues to expose standard Redis metrics only.