-
Notifications
You must be signed in to change notification settings - Fork 1.5k
Added AL2023 support for Outpost nodes #8591
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
base: main
Are you sure you want to change the base?
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -757,8 +757,8 @@ func validateNodeGroupBase(np NodePool, path string, controlPlaneOnOutposts bool | |
| } | ||
| return fmt.Errorf("AMI Family %s is not supported - use one of: %s", ng.AMIFamily, strings.Join(SupportedAMIFamilies(), ", ")) | ||
| } | ||
| if controlPlaneOnOutposts && ng.AMIFamily != NodeImageFamilyAmazonLinux2 { | ||
| return fmt.Errorf("only %s is supported on local clusters", NodeImageFamilyAmazonLinux2) | ||
| if controlPlaneOnOutposts && ng.AMIFamily != NodeImageFamilyAmazonLinux2023 { | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I don't have much context on this feature but just creating an AL2 node we only log a warning and still let users create it if they explicitly try to. Are we worried about breaking customers who may rely on AL2 and aren't ready to migrate yet? Just calling this out explicitly so if that is the intention that is fine too. If this is the intention can we be more clear in the PR title that you can no longer create AL2 outpost clusters (will get added to release notes) |
||
| return fmt.Errorf("only %s is supported on local clusters", NodeImageFamilyAmazonLinux2023) | ||
| } | ||
| } | ||
|
|
||
|
|
||
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
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.
causing a build failure, not sure if you need to rebase / can also leave off the generated changes
https://github.com/eksctl-io/eksctl/actions/runs/19619569203/job/56388775476?pr=8591
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.
I can try to rebase