search

mesos / mr-redis

Redis on Apache Mesos
Apache License 2.0
127 stars 32 forks source link

Support VIP for redis instance #34

Open dhilipkumars opened 8 years ago

dhilipkumars commented 8 years ago

While creating redis instances add attributes to support VIP so that service discovery can be made easy. This is related to https://github.com/mesos/mr-redis/issues/33

ChetanBhasin commented 8 years ago

Hi! Is there a status update on this?

dhilipkumars commented 8 years ago

@ChetanBhasin Hi Sorry we know we are terribly behind on schedule. We got pulled into some internal projects and couldn't spend time on this recently. We will try and get upto speed in completing these open issues.

ChetanBhasin commented 8 years ago

Thanks for the update!

dhilipkumars commented 8 years ago

This framework is built on top of 'mesos-go' language binding. This language binding is pending update since some time, for VIP support the protobuf message specification should be upgraded. The current spec does not cover 'Labels' portion which is important in setting VIP. https://github.com/mesos/mesos-go/pull/263/files#diff-a9294dbaba0cf94ca6b1772f84ca5e9bR537. Long story short for us to be able to use VIP these changes to be pulled into master https://github.com/mesos/mesos-go/pull/263.

eddyzags commented 7 years ago

Hey! What about the service port ? Is there any way to define the service port of the redis master instance in order to contact it with marathon-lb ? Example: Definition: portMappings: [ { "containerPort": 6380, "hostPort": 0, "servicePort": 10001 } ]

Reachable with: marathon-lb.marathon.mesos:10001