search

kuzzleio / kuzzle-plugin-cluster

Kuzzle cluster mode
Apache License 2.0
7 stars 2 forks source link

Nodes can get the same uuid and all think they are master #37

Closed benoitvidis closed 6 years ago

benoitvidis commented 7 years ago

issue is coming from the binding if not resolved.

For instance, [idontexist.ipv4] won't be resolved and the string is kept as is. As all nodes then subscribe to the same private room, they are told confusing information.

ballinette commented 7 years ago

related issue: https://github.com/kuzzleio/kuzzle-private-backlog/issues/68

ballinette commented 7 years ago

Ok, if we set [idontexist:ipv4] (with a ":"), the cluster plugin will try to match the pattern to an existing interface, and fail => the backend will not be registered, as expected.

The issue is if we set anything else, ie for example [idontexits.ipv4] , but also hardip:port, or mydummyuid, then the load balancer will have the same uuid for each node, and declare each one as master. The issue is in Load Balancer, not in plugin-cluster.

ballinette commented 7 years ago

maybe we could just forbid here binding without : character btw the brackets ?