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

AWSMachinePool controller should clean up unused Launch Template Versions #2368

Closed
dlipovetsky opened this issue Apr 22, 2021 · 4 comments · Fixed by #2525
Closed

AWSMachinePool controller should clean up unused Launch Template Versions #2368

dlipovetsky opened this issue Apr 22, 2021 · 4 comments · Fixed by #2525
Labels
kind/bug Categorizes issue or PR as related to a bug.

Comments

@dlipovetsky
Copy link
Contributor

/kind bug

Describe the solution you'd like
The AWSMachinePool controller creates new Launch Template Versions when the configuration changes, or the MachinePool is scaled. (See #2354). AWS imposes a limit on the number of Launch Template Versions:

You are limited to creating 5,000 launch templates per Region and 10,000 versions per launch template.
-- https://docs.aws.amazon.com/AWSEC2/latest/UserGuide/ec2-launch-templates.html#launch-template-restrictions

When this limit is reached, further updates are not possible, and therefore further configuration changes or scale operations would be blocked.

@dlipovetsky
Copy link
Contributor Author

I'm worried that I did not raise this in the v0.6.6 release notes. I think that there's a new version created every time a new bootstrap token is created; assuming that's about every 10m, then a pool can operate for about 10m * 10,000 versions = 100,000m, or approximately 69 days, before its launch template reaches the maximum allowed versions.

@sedefsavas
Copy link
Contributor

sedefsavas commented May 27, 2021

Would you like to address the cleanup in the v0.6.7 release?

I will add this as a warning to the v0.6.6 for new users. (update: added)

@dlipovetsky
Copy link
Contributor Author

I will add this as a warning to the v0.6.6 for new users. (update: added)

Thank you, @sedefsavas!

Would you like to address the cleanup in the v0.6.7 release?

I will try.

@k8s-ci-robot
Copy link
Contributor

@dlipovetsky: You must be a member of the kubernetes-sigs/cluster-api-provider-aws-maintainers GitHub team to set the milestone. If you believe you should be able to issue the /milestone command, please contact your Cluster API Provider AWS Maintainers and have them propose you as an additional delegate for this responsibility.

In response to this:

/milestone v0.7

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Categorizes issue or PR as related to a bug.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants