Skip to content

Conversation

@MyMirelHub
Copy link
Contributor

@MyMirelHub MyMirelHub commented Nov 21, 2025

Thank you for helping make the Dapr documentation better!

Please follow this checklist before submitting:

  • Commits are signed with Developer Certificate of Origin (DCO - learn more)
  • Read the contribution guide
  • Commands include options for Linux, MacOS, and Windows within tabpane
  • New file and folder names are globally unique
  • Page references use shortcodes instead of markdown or URL links
  • Images use HTML style and have alternative text
  • Places where multiple code/command options are given have tabpane

In addition, please fill out the following to help reviewers understand this pull request:

Description

Need to clarify across all redis components that:

  • sentinel supports multiple host addresses
  • sentinel needs failover enabled and redisType node

Issue reference

#4947

…ation

Signed-off-by: MyMirelHub <15373565+MyMirelHub@users.noreply.github.com>
@MyMirelHub MyMirelHub force-pushed the redis-sentinel-multiple-addresses branch from f84a111 to 06637d5 Compare November 21, 2025 10:07
…sage

Signed-off-by: MyMirelHub <15373565+MyMirelHub@users.noreply.github.com>
- name: failover
value: "true"
- name: sentinelMasterName
value: "mymaster"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
value: "mymaster"
value: "sentinelMaster:26379"

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this one isn't a host, - its just the name of the mater set, (see docs)

…disHost should contain sentinel addresses when enabled

Signed-off-by: MyMirelHub <15373565+MyMirelHub@users.noreply.github.com>
…over settings for high availability

Signed-off-by: MyMirelHub <15373565+MyMirelHub@users.noreply.github.com>
…treams setup

Signed-off-by: MyMirelHub <15373565+MyMirelHub@users.noreply.github.com>
…larity

Signed-off-by: MyMirelHub <15373565+MyMirelHub@users.noreply.github.com>
…resses in redisHost for failover configuration

Signed-off-by: MyMirelHub <15373565+MyMirelHub@users.noreply.github.com>
…dancy in sentinel addresses

Signed-off-by: MyMirelHub <15373565+MyMirelHub@users.noreply.github.com>
Signed-off-by: MyMirelHub <15373565+MyMirelHub@users.noreply.github.com>
@MyMirelHub MyMirelHub marked this pull request as ready for review November 21, 2025 15:08
@MyMirelHub MyMirelHub requested review from a team as code owners November 21, 2025 15:08
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