-
Notifications
You must be signed in to change notification settings - Fork 23
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 nodes in different regions #18
Comments
Thanks for the feature request. I am currently carrying out some basic code refactorings. After that it should be easier to realize the feature. |
As far as I tried to implement this feature, I think this will be a breaking change. Changing the datacenter of existing nodes will destroy an recreate them, this is something I expect. I will check how this will behave if primary ips are created with their own resource. Unrelated to the creation of primary ips, changing the datacenter will probably result in different primary ips for all recreated nodes. This enhancement probably needs a bit more research in context with autoscaling, because currently I set the datacenter location for the nodes as a list of strings. With terraform 1.9 there should be a feature introduced to validate a variable condition with external variables. Currently, the list of strings could be longer, or shorter. |
This module is currently not able to carry out meaningful upgrades anyway. Changing the machine config would always lead to a recreate of the server. (I often create my cluster from scratch. Simple/automatic importing of backups helps a lot.) If updates are made, then they should be made via patches. But this is not so easy, because patches are not idempotent. They build on the current state and change it. This means that a patch can have a different effect the second time you run it. I think at the beginning it is sufficient to define several node groups in regions when creating the cluster, similar to kube-hetzner. |
This way you can have a control plane node in each location for europe. (FSN, NBG, HEL)
The text was updated successfully, but these errors were encountered: