Martynas Pumputis comments

Results 164 comments of


Martynas Pumputis

generic-veth chaining not compatible with BPF host routing

@ysksuzuki Sure! Do you plan to fix the issue, or just disable the BPF host routing when the veth chaining is used?

generic-veth chaining not compatible with BPF host routing

> Do you think we should leave a chance for generic-veth users to take advantage of the BPF host routing or just fall back to the legacy host routing? cc...

bpf/nodeport: avoid calling tail call for a DROP_INVALID

Considering https://github.com/cilium/cilium/pull/20425, this PR might be not needed. Moving to draft for now.

bpf/nodeport: avoid calling tail call for a DROP_INVALID

I'd suggest to wait with this PR until https://github.com/cilium/cilium/pull/19712 has been merged. Marking as draft for now.

CI: K8sDatapathConfig Check BPF masquerading with ip-masq-agent VXLAN

Fixed by https://github.com/cilium/cilium/pull/21337.

CI: K8sDatapathConfig Check BPF masquerading with ip-masq-agent VXLAN

Took another stab (cc @squeed): ``` time="2022-09-21T13:30:55Z" level=debug msg="running command: kubectl -n 202209211330k8sdatapathconfigcheckbpfmasqueradingwithip-masq-ag get pods -l zgroup=testDSClient -o jsonpath='{range .items[*]}{@.metadata.name}{\"=\"}{@.status.podIP}{\"\\n\"}{end}'" cmd: "kubectl -n 202209211330k8sdatapathconfigcheckbpfmasqueradingwithip-masq-ag get pods -l zgroup=testDSClient -o jsonpath='{range...

CI: replace usage of PodIP with PodIPs

Thanks @slayer321. Forgot to add `-i` when grepping. The new list: ``` k8s/custom_calls.go: podApp1.Status.PodIP, identityKey, IngressIPv4, k8s/custom_calls.go: podApp1.Status.PodIP, identityKey, EgressIPv4, k8s/datapath_configuration.go: srcPodIP, err := srcPodJSON.Filter("{.status.podIP}") k8s/datapath_configuration.go: dstPodIP, err := dstPodJSON.Filter("{.status.podIP}")...