Re: [ovs-dev] [PATCH ovn v2 3/3] ofctrl: Introduce ecmp_nexthop_monitor.

2024-04-15 Thread Lorenzo Bianconi
> Hi Lorenzo, Hi Mark, thx for the review. > > The code looks fine to me, but I'm a bit confused by the new test. > > My understanding of the new feature is that each ECMP nexthop has an ID > associated with it. This ID gets placed in the ct.label. If the ECMP route > is removed, then we can

Re: [ovs-dev] [PATCH ovn v2 3/3] ofctrl: Introduce ecmp_nexthop_monitor.

2024-04-04 Thread Mark Michelson
Hi Lorenzo, The code looks fine to me, but I'm a bit confused by the new test. My understanding of the new feature is that each ECMP nexthop has an ID associated with it. This ID gets placed in the ct.label. If the ECMP route is removed, then we can find the associated ID, and remove the

[ovs-dev] [PATCH ovn v2 3/3] ofctrl: Introduce ecmp_nexthop_monitor.

2024-03-12 Thread Lorenzo Bianconi
Introduce ecmp_nexthop_monitor in ovn-controller in order to track and flush ecmp-symmetric reply ct entires when requested by the CMS (e.g removing the related static routes). Signed-off-by: Lorenzo Bianconi --- controller/ofctrl.c | 101 ++ controller/ofctrl.h |