On Mon, May 19, 2025 at 10:24 AM Dumitru Ceara <dce...@redhat.com> wrote: > > We recently got an internal request to backport to branch-24.03 (LTS) > the feature that Numan implemented in 24.09.0 that adds support for > centralized routing for distributed ports [0]. > > While in general features should not be backported to older stable > branches, in this case the CMS (OpenStack) is hitting an issue (a bug > from their perspective) which cannot be fixed without OVN support for > centralized routing. > > The new support, introduced by [0] is guarded by an explicit NB > configuration, NB.Logical_Router_Port.options:centralize_routing. > > With this in mind, it seems acceptable to me to also backport the > feature to the LTS (branch-24.03). > > Along with it we also need to backport some prerequisite bugfix (these > were inadvertently not backported to LTS when they were accepted), some > refactor and some follow up patches. > > Aside for some minor conflicts in patch 01/11 due to features that are > only present in 24.09+ the rest of the patches cherry picked by this > series applied (almost) cleanly - some test adjustments had to be made. > > Reported-at: https://issues.redhat.com/browse/FDP-1417 > > [0] https://github.com/ovn-org/ovn/commit/8d13579 > > Ihar Hrachyshka (6): > northd: Don't cleanup op in ovn_port_allocate_key. > northd: Don't detach op->list when it wasn't used. > tests: Correct tunnel ids exhaustion scenario. > northd: Don't create pb in ls_port_init too early. > northd: Remove unused `sb` arg in ls_port_create. > northd: Remove unused nbrp arg in ls_port_reinit. > > Numan Siddique (4): > northd: Don't reparse lport's addresses while adding L2_LKUP flows. > northd: Refactor chassisresident port checking. > Add support for centralize routing for distributed gw ports. > northd: Fix potential crash when creating chassisredirect port. > > Xavier Simonart (1): > multinode: Fix test "ovn multinode NAT ...".
No objections from me for the backports. For the entire series: Acked-by: Numan Siddique <num...@ovn.org> Numan > > NEWS | 3 + > controller/physical.c | 4 + > northd/northd.c | 504 ++++++++++++++++++++++------------ > northd/northd.h | 11 +- > ovn-nb.xml | 34 +++ > tests/multinode-macros.at | 2 +- > tests/multinode.at | 181 +++++++++++++ > tests/ovn-northd.at | 553 +++++++++++++++++++++++++++++++++++++- > tests/ovn.at | 8 +- > 9 files changed, 1110 insertions(+), 190 deletions(-) > > -- > 2.49.0 > _______________________________________________ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev
