-
Notifications
You must be signed in to change notification settings - Fork 1.5k
support Al2023 on outposts #8599
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 Al2023 on outposts #8599
Conversation
AL2 is on deprecation path, making AL2023 the only supported operating system for node-groups on Outpost. Using AL2 will now cause validation error. Leaving the AL2 code in place, to be deleted after this OS is officially deprecated.
|
Hello NicholasBlaskey 👋 Thank you for opening a Pull Request in |
nkvetsinski
left a comment
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.
looks good to me
sapphirew
left a comment
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
| CertificateAuthority: clusterStatus.CertificateAuthorityData, | ||
| CIDR: serviceCIDR, | ||
| } | ||
| if m.cfg.IsControlPlaneOnOutposts() { |
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.
Description
Allow using outposts on AL2023 nodes to give an option after AL2 deprecation notice
continuing this PR #8591
Checklist
README.md, or theuserdocsdirectory)area/nodegroup) and kind (e.g.kind/improvement)BONUS POINTS checklist: complete for good vibes and maybe prizes?! 🤯