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

Fix Galera bootstrap issues #45

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

steventwheeler
Copy link

This fixes a few issues I had when bootstrapping a new Galera cluster.

  1. A race condition exists when managing the users and database on all nodes. This change restricts these actions to the primary node.
  2. I was unable to perform IST without first configuring SST authentication.
  3. MariaDB is not restarted on the second and third nodes after configuring Galera.

@HanXHX
Copy link
Owner

HanXHX commented May 10, 2022

I don't understand your PR.

The good way to use this role: launch playbook on first "bootstrap" node and on a second time launch this role on other nodes.

Did you try to launch this role on all nodes at the same time?

@steventwheeler
Copy link
Author

That explains why I struggled getting it up and running. I was running the playbook against all three nodes in my inventory. Is there a reason that two separate runs are preferred?

@HanXHX
Copy link
Owner

HanXHX commented May 10, 2022

That explains why I struggled getting it up and running. I was running the playbook against all three nodes in my inventory. Is there a reason that two separate runs are preferred?

This Ansible role is very complicated. If you need 'all at the same time', you must put a shared lock system on the first node and wait the bootstrap before executing other tasks...

I don't know how to perform this lock and keep running on the CI 🤣

But I can write a better doc on this setup!

@steventwheeler
Copy link
Author

I'm not sure I follow. What do you mean by "shared lock system"? With the changes in this merge request it is able to bootstrap all three nodes in a single run. Also, regarding the CI failure that appears to be a yamllint issue on a line I didn't change.

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.

2 participants