-
Notifications
You must be signed in to change notification settings - Fork 43
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
Impossible to remove worker groups as described in readme.md #305
Comments
Sorry I linked up the wrong conditional and didn't even read what I pasted in. |
Were u trying to switch between self managed eks worker nodes and eks managed nodes? Setting enable_worker_group to true will provision self managed nodes (https://docs.aws.amazon.com/eks/latest/userguide/worker.html) and setting it to false will provision eks managed node groups (https://docs.aws.amazon.com/eks/latest/userguide/managed-node-groups.html) |
Yes except launch templates will only be added if you set worker groups =true . |
I started writing terragrunt tests for this, but we dont have EKS infrastructure to test this atm 😬 https://github.com/jenkins-x/terraform-aws-eks-jx/tree/master/test |
Summary
The documentation describes that you can disable worker groups. However, if you try to do that you will no longer have worker groups launch templates due to the logic.
The documentation states
However in the code there is a conditional that only allows worker groups launch templates if you also have enabled worker groups
https://github.com/jenkins-x/terraform-aws-eks-jx/blob/master/modules/cluster/main.tf#L99
worker_groups = var.enable_worker_group && !var.enable_worker_groups_launch_template ? [
I'm not quite sure what the correct approach here would be. But the documentation is misleading, and the config is confusing :)
The text was updated successfully, but these errors were encountered: