Skip to content
This repository has been archived by the owner on May 3, 2022. It is now read-only.

Use a custom service name for Hyperbahn ringpop #1394

Open
prashantv opened this issue Oct 19, 2015 · 1 comment
Open

Use a custom service name for Hyperbahn ringpop #1394

prashantv opened this issue Oct 19, 2015 · 1 comment

Comments

@prashantv
Copy link
Contributor

A ringpop application joined the Hyperbahn cluster. It used ringpop as the service name for internal gossip, but due to lack of library support, it ended up talking to Hyperbahn's ringpop.

In this case, it only received one unknown request (destined for a stooge) which it declined, but this seems too dangerous. We should be using a service name like hyperbahn-ringpop so that any bugs or misconfigurations in other applications don't affect Hyperbahn.

@jcorbin @Raynos

@Raynos
Copy link
Contributor

Raynos commented Oct 19, 2015

This should not happen.

Ringpop has a "app" name that it uses in the join() request.

Ringpop-node should reject ringpop-go membvers for foo-app to join.

I'd rather keep the canonical "ringpop" name and add some other kind of protection like changing the name of the ringpop "app" to a full gitolite remote.

To be able to do this extra defensive check we might need to extend ringpop to allow multiple apps during a deploy and use two deploys; one to add a gitolite remote based name and a second to remove the old hyperbahn app name.

cc @jwolski

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants