Add Redis TLS variables to example inventories#339
Merged
steven-schattenberg-itential merged 15 commits intoMay 26, 2026
Merged
Add Redis TLS variables to example inventories#339steven-schattenberg-itential merged 15 commits into
steven-schattenberg-itential merged 15 commits into
Conversation
…nd Sentinel - Remove stale redis_tls_dir/redis_tls_*_file vars from redis.yml defaults that conflicted with pki.yml; template now uses _dest path variables for full paths - Remove superseded TLS block with TODO comment from configure-redis.yml; cert copying is handled by configure-redis-tls.yml - Wire configure-sentinel-tls.yml into main.yml for sentinel nodes when TLS enabled - Restrict configure-redis-tls.yml to data nodes (redis_is_data_node) - Fix redis.conf.j2: use _dest path vars, use redis_tls_auth_clients variable, fix tls-replication condition from undefined redis_replication_enabled to redis_has_replicas - Fix sentinel.conf.j2: add port 0 / tls-port, tls-auth-clients no, and correct _dest path vars per Confluence Redis TLS spec - Add platform_redis_tls_for_sentinel_mode_enabled and platform_redis_sentinel_tls defaults and corresponding properties template lines for Sentinel+TLS support Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
steven-schattenberg-itential
requested review from
Amunagala-itential,
Madias2222,
Nick-Andreano and
kvelarde-itential
kvelarde-itential
changed the title
kvelarde-itential
changed the title
Nick-Andreano
approved these changes
May 26, 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
2 participants
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.
Updated example inventory files