dilyevsky

> Then that could be combined with the new dynamic reflection capabilities in go-protobuf master. So I was looking more deeply into this option and this should work so my...

Maybe proto.Clone from gogo package would still work here? On Wed, Dec 9, 2020 at 6:40 PM John Millikin wrote: > This isn't something Skycfg can fix directly, but when...

Hey John, hope everything is Ok. I should be able to burn some cycles on this comes q1 since we are stuck on older skycfg and proto v1.3.2 due to...

It seems like in order to be able to use the whole library needs to be converted to use new api and its https://google.golang.org/protobuf/types/dynamicpb

> I don't think we want to add Kubernetes API group directly into the protobuf handling, but it might be worth adding a skycfg/kubernetes package to hold all this k8s-specific...

Just to be more clear - the stems from JSON `inline` tag not go type embedding.

Actually I found that standard lib `encoding/json` works as desired by noting the field is "anonymous" (embedded). Looks like this should be an issue to `gogo/protobuf/jsonpb` instead.

Same problem on EKS 1.25 - no NLB is visible in the console and no logs from kcm/cloud-controller beside this event: ``` Normal EnsuringLoadBalancer 6m26s (x2 over 7m30s) service-controller Ensuring...

@rogerson96 try these (i was able to get it to work after reading the controller implementation code): ``` service.beta.kubernetes.io/aws-load-balancer-type: nlb service.beta.kubernetes.io/aws-load-balancer-nlb-target-type: ip ``` TBH the current state of AWS controller...

@rogerson96 pod IP routing only supported if you run native AWS CNI plugin (https://github.com/aws/amazon-vpc-cni-k8s). If you use something like calico or cilium it will still use instance.