Skip to content
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

Seems to not work with a redis cluster. #33

Open
PhillippOhlandt opened this issue Feb 13, 2017 · 1 comment
Open

Seems to not work with a redis cluster. #33

PhillippOhlandt opened this issue Feb 13, 2017 · 1 comment
Labels

Comments

@PhillippOhlandt
Copy link

PhillippOhlandt commented Feb 13, 2017

Please see the issue I opened here: redis/ioredis#431

Has someone tested it on their own and can confirm that it's broken/not broken?

Just to have all the information complete, here are the relevant docker-compose services:

    redis-primary:
        image: bitnami/redis:latest
        environment:
            - REDIS_REPLICATION_MODE=master
        volumes:
            - ./_redisdata:/bitnami/redis
        restart: always

    redis-secondary:
        image: bitnami/redis:latest
        depends_on:
            - redis-primary
        environment:
            - REDIS_REPLICATION_MODE=slave
            - REDIS_MASTER_HOST=redis-primary
            - REDIS_MASTER_PORT=6379
        restart: always

And here is the DS config:

plugins:
  message:
    name: redis
    options:
      nodes:
       - host: "redis-primary"
         port: 6379
       - host: "redis-secondary"
         port: 6379
      maxRedirections: 16
      scaleReads: slave
      redisOptions:
        foo: bar # just to have an entry here, otherwise the connector will fail in another way
  cache:
    name: redis
    options:
      nodes:
       - host: "redis-primary"
         port: 6379
       - host: "redis-secondary"
         port: 6379
      maxRedirections: 16
      scaleReads: slave
      redisOptions:
        foo: bar
@PhillippOhlandt
Copy link
Author

I created a little demo project to show the failure. https://github.com/PhillippOhlandt/deepstream-redis-cluster

@yasserf yasserf added the in qa label Jul 31, 2017
@AlexBHarley AlexBHarley added ready and removed in qa labels Aug 25, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants