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

support for clustering by hostname in kubernetes #13

Merged
merged 2 commits into from Apr 10, 2017

Conversation

pavel987
Copy link

Copying my pull request from aweber#118

@@ -56,6 +56,8 @@
{config, k8s_namespace_path, "K8S_NAMESPACE_PATH", "/var/run/secrets/kubernetes.io/serviceaccount/namespace",
string, false},
{config, k8s_service_name, "K8S_SERVICE_NAME", "rabbitmq", string, false},
{config, k8s_address_type, "K8S_ADDRESS_TYPE", "ip", string, false},
Copy link
Member

Choose a reason for hiding this comment

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

If the options here are ip and hostname, please leave a comment mentioning that.

Copy link
Member

Choose a reason for hiding this comment

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

Actually, the rest looks good so I will add one myself.

@michaelklishin michaelklishin merged commit aee51d8 into rabbitmq:master Apr 10, 2017
@michaelklishin
Copy link
Member

@pavel987 thank you!

@luxflux
Copy link

luxflux commented May 2, 2017

Thank you for your work, @pavel987 and @michaelklishin. What do you think could we have this also for RabbitMQ v3.6.x?

@michaelklishin
Copy link
Member

@Gsantomaggio @dcorbacho does this seem like something you'd agree to backport?

@luxflux
Copy link

luxflux commented May 3, 2017

IMHO this is crucial to get a cluster on k8s with persistence. If using IPs, the cluster will loose it's database (incl. users, ...) sooner or later because the IPs change if a pod gets stopped/started.
But it could as well be that I did something wrong when configuring it...

I got the code working on the stable branch and it seems to work. Should I provide a PR...?

@Gsantomaggio
Copy link
Member

Yes!

@michaelklishin
Copy link
Member

@luxflux please do.

luxflux added a commit to luxflux/rabbitmq-autocluster that referenced this pull request May 8, 2017
luxflux added a commit to luxflux/rabbitmq-autocluster that referenced this pull request May 8, 2017
@luxflux
Copy link

luxflux commented May 8, 2017

PR #23 ports the changes of @pavel987 to stable.

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.

None yet

4 participants