terraform-aws-eks icon indicating copy to clipboard operation
terraform-aws-eks copied to clipboard

Attribute-Based Instance Type Selection for node groups

Open youwalther65 opened this issue 3 years ago β€’ 10 comments

Is your request related to a new offering from AWS?

Yes, see: https://aws.amazon.com/blogs/aws/new-attribute-based-instance-type-selection-for-ec2-auto-scaling-and-ec2-fleet/ https://docs.aws.amazon.com/autoscaling/ec2/userguide/create-asg-instance-type-requirements.html

The feature is not yet implemented in the Terraform AWS provider, see corresponding issue: https://github.com/hashicorp/terraform-provider-aws/issues/21566

This would be a great feature for node groups using SPOT or ON DEMAND with instance diversification. It would simplify node group configuration and make it future proof by just provision the needed resources (for example vCPU and memory) and let the corresponding AWS API pickup suitable instance types even new ones as they become available.

youwalther65 avatar Jan 19 '22 19:01 youwalther65

This issue has been automatically marked as stale because it has been open 30 days with no activity. Remove stale label or comment or this issue will be closed in 10 days

github-actions[bot] avatar Feb 20 '22 00:02 github-actions[bot]

This issue was automatically closed because of stale in 10 days

github-actions[bot] avatar Mar 03 '22 00:03 github-actions[bot]

bad bot

bryantbiggs avatar Mar 03 '22 01:03 bryantbiggs

This issue has been automatically marked as stale because it has been open 30 days with no activity. Remove stale label or comment or this issue will be closed in 10 days

github-actions[bot] avatar Apr 03 '22 00:04 github-actions[bot]

This issue was automatically closed because of stale in 10 days

github-actions[bot] avatar Apr 13 '22 00:04 github-actions[bot]

Would a maintainer please reopen and stop the bot from marking this stale?

risinger avatar May 10 '22 20:05 risinger

Launch template support has been added to the AWS provider, we still need autoscaling group support as well https://github.com/hashicorp/terraform-provider-aws/issues/21566

bryantbiggs avatar May 16 '22 14:05 bryantbiggs

@bryantbiggs I believe https://github.com/hashicorp/terraform-provider-aws/issues/21566 doesn't cover the aws_eks_node_group resource unless I have missed something. :-)

May be we look at using ec2_instance_types data source to implement instance selector similar to how eksctl does here

Any thoughts?

sacksi28 avatar May 27 '22 15:05 sacksi28

Correct - getting it into the EKS managed node group is a whole different ball of yarn. First we need the EKS service to support it in managed node groups, then the AWS provider will need to update (possibly, depending on implementation/API). We could try passing a launch template with the attribute based instance selection and see what happens - haven't tried that myself 🀷🏽

bryantbiggs avatar May 27 '22 16:05 bryantbiggs

https://github.com/hashicorp/terraform-provider-aws/issues/21566 is now closed so work on this can progress.

ruecarlo avatar Sep 22 '22 15:09 ruecarlo

This feature would be very useful. Can someone look into it?

davidweizheng avatar Oct 21 '22 14:10 davidweizheng

v19 needs to be released to increase the provider version to at least v4.x first

bryantbiggs avatar Oct 21 '22 14:10 bryantbiggs

This issue has been resolved in version 19.0.0 :tada:

antonbabenko avatar Dec 05 '22 21:12 antonbabenko

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[bot] avatar Jan 05 '23 02:01 github-actions[bot]