Acked-by: Mark Michelson <[email protected]>

I'm fine with getting this change in for 21.09 with the intent of expanding on I-P for upcoming releases.

On 9/16/21 11:49 AM, Mark Gray wrote:
These commits reorganise the northd code base in preparation for potential 
inclusion
of I-P for northd.

Please note that this mainly involves moving code around with minimal
code changes. However, due to tight coupling between ovn-northd.c and northd.c,
some minor changes were needed. For reference, and to help reviews, please
examine the following at a minimum:

* Configuration of the probe interval in northd.c 
(ovsdb_idl_set_probe_interval())
* Passing of "use_parallel_build" and "lflow_locks" from ovn-northd.c and
   northd.c.
* Update of "struct northd_context": additon of fields and move to .h file.

The commits were (hopefully) structured in a way to make the review easier. As
this change touches all of ovn-northd, any change to "master" will make a rebase
necessary and probably difficult. Therefore, if the general ideas is OK, then
it would be great if this series could be expedited to prevent many rebases!

Thanks

---
v2: Rebase
v3: Rebase. Fixed compile-time error
v4: Add additional commits which add framework for incremental processing in 
northd
v5: Remove additional IP commits 
(https://mail.openvswitch.org/pipermail/ovs-dev/2021-September/387763.html)
     Rebase

Mark Gray (2):
   ovn-northd: Rename ovn-northd.c to northd.c
   northd: Split northd.c

  Documentation/tutorials/ovn-openstack.rst |   154 +-
  northd/automake.mk                        |     2 +
  northd/lrouter.dl                         |     2 +-
  northd/northd.c                           | 14579 +++++++++++++++++++
  northd/northd.h                           |    42 +
  northd/ovn-northd.c                       | 14880 +-------------------
  northd/ovn.rs                             |     2 +-
  northd/ovn_northd.dl                      |     2 +-
  tests/ovn-northd.at                       |     2 +-
  9 files changed, 14871 insertions(+), 14794 deletions(-)
  create mode 100644 northd/northd.c
  create mode 100644 northd/northd.h


_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to