-
Notifications
You must be signed in to change notification settings - Fork 4k
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 AWS "Instance type requirements" #4479
Comments
/area provider/aws Hey, thanks for the suggestion. When raising feature requests like this, it helps us a huge amount if you fill out the answers to the questions in the feature request issue template including linking to any relevant external documentation. |
Hey @gjtempleton, it would help a lot if you can point me to the lines of code that handle the following logics:
Thanks! |
@garrygerber thank you for opening the issue! I've submitted a PR to help address this issue. |
The Kubernetes project currently lacks enough contributors to adequately respond to all issues and PRs. This bot triages issues and PRs according to the following rules:
You can:
Please send feedback to sig-contributor-experience at kubernetes/community. /lifecycle stale |
The Kubernetes project currently lacks enough active contributors to adequately respond to all issues and PRs. This bot triages issues and PRs according to the following rules:
You can:
Please send feedback to sig-contributor-experience at kubernetes/community. /lifecycle rotten |
The Kubernetes project currently lacks enough active contributors to adequately respond to all issues and PRs. This bot triages issues and PRs according to the following rules:
You can:
Please send feedback to sig-contributor-experience at kubernetes/community. /close |
@k8s-triage-robot: Closing this issue. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
/reopen |
@garrygerber: Reopened this issue. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
Good news @garrygerber - a PR was just accepted a few days ago for supporting Instance Requirements: fed54b5 |
Which component are you using?:
Cluster-Autoscaler
Is your feature request designed to solve a problem? If so describe the problem this feature should solve.:
AWS added a new way of specifying instance types wanted.
they do this by specifying a set of instance characteristics - requirements.
which obsolete the need of specifying instance types, and in turn, breaks the CA when fetching the ASG info
Describe the solution you'd like.:
CA needs to support not knowing which instance type is used by the ASG
Describe any alternative solutions you've considered.:
CA will reed all the instance requirements from the ASG and calculate which instance type ASG will use when needed
Additional context.:
using CA v1.22.1 , error is:
E1128 15:47:36.224041 1 static_autoscaler.go:286] Failed to get node infos for groups: ASG "......." uses the unknown EC2 instance type ""
new settings in console looks:
The text was updated successfully, but these errors were encountered: