On Thu, Feb 20, 2025 at 02:47:24PM +0100, Dumitru Ceara wrote: > On 2/18/25 3:02 PM, Felix Huettner via dev wrote: > > In this series we add support for incremental processing of learned > > routes. At the end of the series adding or removing learned routes no > > longer triggers an lflow recompute. > > > > Also the logic in here supports other types of routes too. We there > > however need to add incremental support to the route engine node. > > > > This patchset includes > > https://patchwork.ozlabs.org/project/ovn/patch/408bc00c8a1158e4fdf5b1afd4ac0a816744a615.1739462687.git.felix.huettner@stackit.cloud/ > > to prevent conflicts. > > > > Felix Huettner (6): > > tests: Fix flanky I+P tests. > > northd: Add en-learned-route-sync I+P handling. > > northd: Add group ecmp route engine node. > > northd: Support I+P for group_ecmp_route engine. > > northd: Add lflow_ref for route entries. > > northd: Support I+P of lflows on route changes. > > > > Hi Felix, > > Thanks for the series! > > It seems correct to me overall. I did leave some small comments on the > individual patches.
Hi Dumitru, thanks a lot. > > It also gives me the impression that this is the first northd > incremental processing series in a while that didn't require an immense > amount of effort to write. That's a good thing, I guess! :) Yes it was mostly focusing on the actual incremental logic. Splitting the engine nodes went quite well. But i am not sure if adding the same for static routes would be as easy :) Thanks, Felix > > Regards, > Dumitru > _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
