Skip to content

Conversation

@ayruslore
Copy link
Contributor

@ayruslore ayruslore commented Jul 18, 2025

Add support for ipv6 target groups.

Also bases the changes from #751

The PRs on k8s-on-aws zalando-incubator/kubernetes-on-aws#9627 and zalando-incubator/kubernetes-on-aws#9637

AlexanderYastrebov and others added 8 commits July 17, 2025 14:04
CloudFormation library https://github.com/mweagle/go-cloudformation
was last updated in October 2021 and recommends to use https://github.com/awslabs/goformation
which is archived since October 2024.

This change vendors in this dependency to enable addition of new missing
CloudFormation features.

* clone github.com/mweagle/go-cloudformation into aws/cloudformation
* remove unused aws/cloudformation/deploycfn
* replace imports

Updates #459

Signed-off-by: Alexander Yastrebov <[email protected]>
Signed-off-by: Alexander Yastrebov <[email protected]>
Signed-off-by: Alexander Yastrebov <[email protected]>
Signed-off-by: Alexander Yastrebov <[email protected]>
Signed-off-by: Alexander Yastrebov <[email protected]>
Signed-off-by: Alexander Yastrebov <[email protected]>
Signed-off-by: speruri <[email protected]>
@ayruslore
Copy link
Contributor Author

We need the load balancer ip address type to be dual stack so that we can add ipv6 target groups.

See error message from cf stack

This AWS::ElasticLoadBalancingV2::Listener resource is in a UPDATE_FAILED state.

Resource handler returned message: "You cannot associate an IPv6 target group with a load balancer that has an 'ipv4' IP 
address type (Service: ElasticLoadBalancingV2, Status Code: 400, Request ID: 1c8e8ce0-eefa-4ae4-b61a-95e72bca5812) 
(SDK Attempt Count: 1)" (RequestToken: ced9b7e0-78da-123f-803d-f48404feb18c, HandlerErrorCode: InvalidRequest)

@ayruslore ayruslore changed the title Test/eks karp vendor in cf ipv6 - Add support for ipv6 target groups Add support for ipv6 target groups Jul 24, 2025
@ayruslore
Copy link
Contributor Author

Closing in favour of #763

@ayruslore ayruslore closed this Sep 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants