alimehrabikoshki
alimehrabikoshki
@sinkingpoint I see the MR is still open, are you still happy to see that through to completion?
@simonpasquier did you get a chance to think about this one any further?
Hi @bimmlerd I added support for multiple CIDRs to the flag (comma-separated) and changed the BPF implementation to use LPM trie maps. I'd appreciate a review on the way the...
> Are there any known workarounds to achieve this behavior in the meantime? Also, is there an estimated timeline for when this PR might be merged or released? The workaround...
> could you rebase on main and resolve them? Done - Not sure about the new failure on [Wait for lint checks](https://github.com/cilium/cilium/actions/runs/17861759358/job/50794075548?pr=40132#logs) though. Seems [some others are also having the...
I've rebased to resolve conflicts. @gentoo-root and @derailed, do you have any further concerns on this PR?
I rebased again, https://github.com/cilium/cilium/commit/1cd3113371732cca96fee4e5142813e55806d0cd fixed the issue with the remote-node-masquerade tests that I was running into, so I removed my changes to these files from this PR.
> Latest rebase seems to hit some linting problem: https://github.com/cilium/cilium/actions/runs/19507355062/job/55840085730?pr=40132 > > ``` > pkg/datapath/maps/cell.go:1: : # github.com/cilium/cilium/pkg/datapath/maps [github.com/cilium/cilium/pkg/datapath/maps.test] > Error: pkg/datapath/maps/map_test.go:239:13: undefined: NewMapSweeper > Error: pkg/datapath/maps/map_test.go:257:13: undefined: NewMapSweeper (typecheck)...
Thanks @brb and @julianwiedmann for the comments. > Did you consider reusing the whole BPF ip-masq-agent infrastructure to resolve the problem? We could teach the agent to automatically add CiliumPodIPPools...
> Hello, everyone! > > Apologies for interrupting the discussion, but haven't you consider behavior similar to Calico pools? There is option natOutgoing within pool configuration. When enabled, packets sent...