I forgot the "ovn" designation on this series. I've removed it from OVS
patchwork and am re-submitting the series with the "ovn" designation.
Please do not reply to this series, but reply to the next one I post.
On 4/21/25 17:19, Mark Michelson via dev wrote:
This patch series seeks to refactor how northbound datapath and port
types are synced with southbound Datapath_Bindings and Port_Bindings.
In current OVN, the en-northd node is responsible for creating, updating
and deleting all southbound Datapath_Bindings and Port_Bindings. This
means that if you have a new type of Datapath_Binding or Port_Bidning
that you want to add to OVN, it needs to be added to the en-northd node.
An upcoming feature (composable services) will be adding new types of
southbound Datapath_Bindings and Port_Bindings. However, it does not fit
well into the current en_northd environment and would do better
separated into its own set of engine nodes. In order to allow this, the
Datapath_Binding and Port_Binding syncing code needs to be extracted to
separate nodes.
This series does just that. The en_northd node is stripped of its
previous functionality of creating, updating, and deleting southbound
Datapath_Bindings and Port_Bindings. These are now accomplished in
dedicated incremental engine nodes that are inputs to the en_northd
engine node.
Note that the first two versions of this series only had the datapath
refactor present. It did not have the final two patches for port binding
refactoring.
Mark Michelson (5):
en-global-config: Cache vxlan mode and max tunnel key.
northd: Remove lr_list from en_northd data.
northd: Refactor datapath syncing.
ovn.at: Change MAC Bindings used by check packet length test.
northd: Refactor port binding pairing.
TODO.rst | 24 +
northd/automake.mk | 20 +-
northd/datapath_sync.c | 57 +
northd/datapath_sync.h | 77 ++
northd/en-datapath-logical-router.c | 175 +++
northd/en-datapath-logical-router.h | 48 +
northd/en-datapath-logical-switch.c | 176 +++
northd/en-datapath-logical-switch.h | 47 +
northd/en-datapath-sync.c | 319 ++++++
northd/en-datapath-sync.h | 27 +
northd/en-global-config.c | 64 +-
northd/en-global-config.h | 4 +
northd/en-northd.c | 53 +-
northd/en-northd.h | 1 -
northd/en-port-binding-chassisredirect.c | 279 +++++
northd/en-port-binding-chassisredirect.h | 52 +
northd/en-port-binding-logical-router-port.c | 149 +++
northd/en-port-binding-logical-router-port.h | 47 +
northd/en-port-binding-logical-switch-port.c | 201 ++++
northd/en-port-binding-logical-switch-port.h | 48 +
northd/en-port-binding-pair.c | 459 ++++++++
northd/en-port-binding-pair.h | 32 +
northd/inc-proc-northd.c | 92 +-
northd/northd.c | 1058 ++++--------------
northd/northd.h | 26 +-
northd/port_binding_pair.c | 79 ++
northd/port_binding_pair.h | 115 ++
tests/ovn.at | 4 +-
28 files changed, 2824 insertions(+), 909 deletions(-)
create mode 100644 northd/datapath_sync.c
create mode 100644 northd/datapath_sync.h
create mode 100644 northd/en-datapath-logical-router.c
create mode 100644 northd/en-datapath-logical-router.h
create mode 100644 northd/en-datapath-logical-switch.c
create mode 100644 northd/en-datapath-logical-switch.h
create mode 100644 northd/en-datapath-sync.c
create mode 100644 northd/en-datapath-sync.h
create mode 100644 northd/en-port-binding-chassisredirect.c
create mode 100644 northd/en-port-binding-chassisredirect.h
create mode 100644 northd/en-port-binding-logical-router-port.c
create mode 100644 northd/en-port-binding-logical-router-port.h
create mode 100644 northd/en-port-binding-logical-switch-port.c
create mode 100644 northd/en-port-binding-logical-switch-port.h
create mode 100644 northd/en-port-binding-pair.c
create mode 100644 northd/en-port-binding-pair.h
create mode 100644 northd/port_binding_pair.c
create mode 100644 northd/port_binding_pair.h
_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
