Skip to content

Conversation

jschaul
Copy link
Member

@jschaul jschaul commented May 2, 2022

Actually, this redis runs in "master" mode on that port (from docker-compose). There are other redises launched on other parts that run in cluster mode.

Redis part of cluster:
https://github.com/wireapp/wire-server/blob/develop/deploy/dockerephemeral/docker-compose.yaml#L98
(ports 6373 - 6378)

Redis standalone in master mode on port 6379
https://github.com/wireapp/wire-server/blob/develop/deploy/dockerephemeral/docker-compose.yaml#L77

Sorry for forgetting to do this earlier (to add config to this file). The PR #2339 is doing the wrong thing, any use of gundeck and messaging on the demo setup would fail.

Actually, this redis runs in "master" mode on that port (from docker-compose). There are other redises launched on other parts that run in cluster mode.

Redis part of cluster:
https://github.com/wireapp/wire-server/blob/develop/deploy/dockerephemeral/docker-compose.yaml#L98
(ports 6373 - 6378)

Redis standalone in master mode on port 6379
https://github.com/wireapp/wire-server/blob/develop/deploy/dockerephemeral/docker-compose.yaml#L77
@jschaul jschaul requested review from pcapriotti and supersven May 2, 2022 10:41
@jschaul jschaul temporarily deployed to cachix May 2, 2022 10:41 Inactive
Copy link
Contributor

@supersven supersven left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@jschaul jschaul merged commit bc59380 into develop May 2, 2022
@jschaul jschaul deleted the fixup-2339 branch May 2, 2022 12:17
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