aws / aws-k8s-tester

AWS Kubernetes tester, kubetest2 deployer implementation
Apache License 2.0
163 stars 82 forks source link

Remove AMISSMParameter option from unmanaged nodegroup template #415

Closed cartermckinnon closed 9 months ago

cartermckinnon commented 9 months ago

Description of changes:

Even when this CFN template parameter is not required (AMIId is defined), the default value is resolved. This will fail for Kubernetes versions that aren't GA on EKS yet (i.e. the SSM parameter doesn't exist yet).

We intend to always pass an AMI ID to this template, so I'm just removing this template parameter.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.