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 node_group_name_prefix for aws_eks_node_group #16699

Closed
mfreebairn-r7 opened this issue Dec 10, 2020 · 5 comments
Closed

Support node_group_name_prefix for aws_eks_node_group #16699

mfreebairn-r7 opened this issue Dec 10, 2020 · 5 comments
Labels
enhancement Requests to existing resources that expand the functionality or scope. service/eks Issues and PRs that pertain to the eks service.

Comments

@mfreebairn-r7
Copy link

mfreebairn-r7 commented Dec 10, 2020

Community Note

  • Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
  • Please do not leave "+1" or other comments that do not add relevant new information or questions, they generate extra noise for issue followers and do not help prioritize the request
  • If you are interested in working on this issue or have submitted a pull request, please leave a comment

Description

aws_eks_node_group should support node_group_name_prefix.

Currently it only supports node_group_name. This is an issue when you want to change instance type(s) for your node group as terraform forces a replacement of the node group, and because node_group_name must be unique, this therefore prevents you from adding the create_before_destroy lifecycle.

This leads to downtime (zero compute due to the node group having to be destroyed) every time you want to change instance type(s) (even if you are only adding an additional instance type to the instance_types list).

node_group_name_prefix would allow us to support create_before_destroy and therefore we could bring up a new node group prior to destroying the old one when we want to change instance_types

New or Affected Resource(s)

aws_eks_node_group

@mfreebairn-r7 mfreebairn-r7 added the enhancement Requests to existing resources that expand the functionality or scope. label Dec 10, 2020
@github-actions github-actions bot added the needs-triage Waiting for first response or review from a maintainer. label Dec 10, 2020
@Skaronator
Copy link

This seems to be fixed by the following PRs once merged.

@mfreebairn-r7
Copy link
Author

Thanks, I missed those PRs!

@dstevensson
Copy link

@mfreebairn-r7 this was fixed in #13938 which made it's way into v3.40.0. As a workaround before the PR got accepted I started changing the instance type in a custom launch template instead which allowed me to change the instance type without bringing down all the nodes.

@breathingdust breathingdust added service/eks Issues and PRs that pertain to the eks service. and removed needs-triage Waiting for first response or review from a maintainer. labels Sep 16, 2021
@breathingdust
Copy link
Member

This appears to be resolved as the comments indicates. Closing.

@github-actions
Copy link

github-actions bot commented Jun 8, 2022

I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues.
If you have found a problem that seems similar to this, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jun 8, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement Requests to existing resources that expand the functionality or scope. service/eks Issues and PRs that pertain to the eks service.
Projects
None yet
Development

No branches or pull requests

4 participants