Skip to content

/ symfony

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[Messenger] Add Redis sentinel support #41762

Open
hoffmann-andras opened this issue Jun 21, 2021 · 3 comments
Open

[Messenger] Add Redis sentinel support #41762

hoffmann-andras opened this issue Jun 21, 2021 · 3 comments
Labels
Feature Messenger

Comments

@hoffmann-andras
Copy link

@hoffmann-andras hoffmann-andras commented Jun 21, 2021 •
edited

Description
Please, add sentinel support to redis-messenger.

Example

This is how it would be configured (similar to what we do today in Cache component):

.env
MESSENGER_TRANSPORT_DSN='redis:?host[redis1:26379]&host[redis2:26379]&host[redis3:26379]&redis_sentinel=mymaster'
@javiereguiluz
Copy link
Member

@javiereguiluz javiereguiluz commented Jun 22, 2021

@hoffmann-andras you mean that we're missing this from Symfony Docs?

We mention Redis Sentinel in Cache (here: https://symfony.com/doc/current/components/cache/adapters/redis_adapter.html) but not in Messenger (here: https://symfony.com/doc/current/messenger.html#redis-transport). If that's your question, we'll move this to Symfony Docs repository.
@hoffmann-andras
Copy link
Author

@hoffmann-andras hoffmann-andras commented Jun 23, 2021

@javiereguiluz No, it's not a question it's a feature request. Sorry, I mean the description as acceptance criteria.
We would like to use messenger component with redis sentinel, but it has no support, just plain redis and redis cluster.
So the Feature label was correct I think.
@javiereguiluz javiereguiluz changed the title [Messenger] Redis sentinel support [Messenger] Add Redis sentinel support Jun 23, 2021
@javiereguiluz
Copy link
Member

@javiereguiluz javiereguiluz commented Jun 23, 2021

OK thanks! I've updated the issue description a bit.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Feature Messenger
Projects
None yet
Milestone
No milestone
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
3 participants
@javiereguiluz @carsonbot @hoffmann-andras