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.

Sign up for GitHub

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

Jump to bottom

743 rabbitmq cluster #867

Merged
merged 37 commits into from
Feb 27, 2019
Merged

743 rabbitmq cluster #867

merged 37 commits into from
Feb 27, 2019

Conversation

rtprio
Copy link
Contributor

@rtprio rtprio commented Jan 24, 2019

provides .statefulset-lagoon.app.yml to run as cluster.

closes #743

Elliot Schlegelmilch added 13 commits January 10, 2019 14:17
add the plugin for peer discovery #743.
ff11e24
this does establish a cluster.
76c52f5
place config in configmap; place in seperate directory; copy into /et…
126dbf6 
…c/rabbitmq so that rabbitmq will start.

error now is:  Failed to get nodes from k8s - 403.
this plugin, connecting to kube api, needs a service account with suc…
e88e4cb 
…h priviledges.
shorten selectors/service to SERVICE_NAME.
ff5e2bc
typo.
66ba946
erlang cookies need to be the same on all nodes.
3ee2d4f
\ No newline at end of file
393bcb9
you need clusterip to talk to talk between statefulset pods.
32d30d1
this hostname works.
405aafa
add back rabbitmq_delayed_message_exchange now that the /etc/rabbitmq…
4662d07 
… directory is assembled from scratch.
add explict path info for tokens and certs.
a72a87b
dynamically configure nodename. move setup to script, called on spec.
513bcd8
@rtprio rtprio self-assigned this Jan 24, 2019
twardnw
twardnw reviewed Jan 24, 2019
View reviewed changes
Copy link
Contributor

@twardnw twardnw left a comment

Choose a reason for hiding this comment

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

It was mentioned during a standup that some extra permissions were needed in OpenShift for this, can you add that as a comment on the PR?

@rtprio
Copy link
Contributor Author

rtprio commented Jan 24, 2019

Sure;

The plugin calls kubernetes.default.svc http endpoint to fetch a list of containers from the statefulset. In order to do this it needs a serviceaccount with privileges and to run the containers as that service account.

@rtprio
Copy link
Contributor Author

rtprio commented Feb 21, 2019

@Schnitzel can you verify the changes to docker-compose.yaml ?

Schnitzel added a commit that referenced this pull request Feb 26, 2019
@Schnitzel
Merge pull request #867 from amazeeio/743-rabbitq-cluster
b347760 
743 rabbitmq cluster
@Schnitzel Schnitzel merged commit 7481939 into master Feb 27, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@twardnw twardnw twardnw left review comments

@Schnitzel Schnitzel Awaiting requested review from Schnitzel

Assignees

@rtprio rtprio

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

RabbitMQ Cluster
3 participants
@rtprio @twardnw @Schnitzel