wweiwei-li

Results 44 comments of wweiwei-li

Helm update to version 1.7.x resulting in pods Readiness probe failed: HTTP probe failed with statuscode: 404

This is known issue. Can you confirm that you are using latest version for controller also. If you are updating chart version, you need also to use v2.7.0 controller version

Helm update to version 1.7.x resulting in pods Readiness probe failed: HTTP probe failed with statuscode: 404

https://github.com/aws/eks-charts/blob/master/stable/aws-load-balancer-controller/values.yaml. Here is the most recent helm chart, which links to the right controller version. Can you tell us from where did you see the v2.4.6.

Add non-alphanumeric input validation test for vpcID and fix typo

/lgtm

Add support for optionally enforcing NLB security groups on PrivateLi…

Hey, This feature will be released in V2.8.0 targeting the end of this week.

TargetGroupBinding using port instead of nodePort when service is of type NodePort

Hey, the port used in serviceRef should be the port of the service. So in your case, you should set the service port as 3081 Example of how it works....

Issue with Reusing Pre-existing NLB Created via Terraform in Kubernetes Service

We do not support using existing NLB for service now. But we are have a feature soon, You can track it here : https://github.com/kubernetes-sigs/aws-load-balancer-controller/issues/228 TargetGroupBlinding supports Existing ALB/NLB, can you...

End of support for WAF Classic

Thanks for bringing this up. We are aware of this, and will plan on deprecate the support of --enable-waf flag

Add support for extra security groups annotation (NLB)

/lgtm

Add support for extra security groups annotation (NLB)

/retest

Cannot set the IPv6 addresses in dualstack mode during modification

I can reproduce this issue. The reason the specified IPv6 address is not set because we are only [checking subnetID](https://github.com/kubernetes-sigs/aws-load-balancer-controller/blob/e5d625f96415fd44e6399e9c75e2bd985f5a2288/pkg/deploy/elbv2/load_balancer_manager.go#L164) to determine if we need to update SubnetMappings, we also...