Hi Lorenzo, I had a look through this patch series and it looks good to me. The only thing that that I noticed was that in patches 2-5, the added documentation uses "CSM" instead of "CMS" in all XML documentation for new options. I also have one small note on patch 5. With those items addressed:
Acked-by: Mark Michelson <[email protected]> On Tue, May 26, 2026 at 2:50 AM Lorenzo Bianconi via dev <[email protected]> wrote: > > Changes in v2: > - Filter out advertised_mac_binding on northd side. > > Lorenzo Bianconi (5): > northd: Add the capability to select static routes to advertise via > BGP. > northd: Add the capability to select host-IPs to advertise via BGP. > northd: Add the capability to select NAT IP to advertise via BGP. > northd: Add the capability to select LB VIP to advertise via BGP. > controller: Add the capability to select EVPN MAC-IP bindings to > advertise. > > controller/neighbor.c | 4 ++ > northd/en-advertised-route-sync.c | 34 +++++++++++-- > northd/en-learned-route-sync.c | 2 +- > northd/en-lr-stateful.c | 38 +++++++++++++- > northd/lb.c | 22 +++++++- > northd/lb.h | 4 +- > northd/northd.c | 25 +++++++-- > northd/northd.h | 2 + > ovn-nb.xml | 44 +++++++++++++--- > ovn-sb.xml | 9 ++++ > tests/ovn-northd.at | 85 +++++++++++++++++++++++++++++++ > tests/system-ovn.at | 23 +++++++++ > 12 files changed, 272 insertions(+), 20 deletions(-) > > -- > 2.54.0 > > _______________________________________________ > dev mailing list > [email protected] > https://mail.openvswitch.org/mailman/listinfo/ovs-dev > _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
