-
Notifications
You must be signed in to change notification settings - Fork 833
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
Add note to clarify this upgrading is about the OS #1463
Conversation
✔️ Deploy Preview for karpenter-docs-prod ready! 🔨 Explore the source changes: 8e7a34d 🔍 Inspect the deploy log: https://app.netlify.com/sites/karpenter-docs-prod/deploys/62210bc9aa744700081010ee 😎 Browse the preview: https://deploy-preview-1463--karpenter-docs-prod.netlify.app |
@@ -72,7 +72,7 @@ For more details on how Karpenter deletes nodes, see [Deprovisioning nodes](../t | |||
### Upgrading nodes | |||
|
|||
A straight-forward way to upgrade nodes is to set `ttlSecondsUntilExpired`. | |||
Nodes will be terminated after a set period of time and will be replaced with newer nodes. | |||
Nodes will be terminated after a set period of time and will be replaced with newer nodes using the latest [EKS Optimized AMI](https://docs.aws.amazon.com/eks/latest/userguide/eks-optimized-amis.html) or the AMI specified in the `$LATEST` version of your launch template. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
note that it's not just EKSOptimized, it's any AMI (e.g. Bottlerocket)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
AL2, Ubuntu, Bottlerocket, and Windows are "EKS Optimized" AMI according to the link: https://docs.aws.amazon.com/eks/latest/userguide/eks-optimized-amis.html
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm!
1. Issue, if available:
2. Description of changes:
This seems a bit vague, 'upgrade' could be interpreted in different ways including vertically scaling the nodes. This addition makes it clear that upgrading is about upgrading the operating system (i.e. the AMI) and only relevant when using AL2 and Bottlerocket.
3. How was this change tested?
4. Does this change impact docs?
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.