[ovs-dev] [PATCH v3 ovn 0/3] controller: Notify when an OVN port is ready to use.

Dumitru Ceara dceara at redhat.com
Mon Jan 11 12:06:40 UTC 2021


This series adds support to ovn-controller to notify northd that all OVS
flows and OVS interface corresponding to an OVN logical port have been
installed in OVS.

The series extends the ofctrl barrier implementation and makes it generic
so that multiple modules can use it.  For now these are:
- main ovn-controller for nb_cfg propagation.
- binding module for marking Port_Bindings as installed.

V3:
- Address Mark's comments:
  - Patch2:
    - Remove unnecessary ENABLE_UNIT_TESTS.
  - Patch3:
    - Add test to check that the ovn-installed external-id is propagated to
      OVS.  The test also checks that the SB Port_Binding.up and
      NB.Logical_Switch_Port.up fields are properly set.
- Patch3:
  - Improve the checks in binding_seqno_install() to cleanup stale bindings
    early.
V2:
- Patch 3:
  - Fix check in binding_seqno_install() to take into account deleted
    port bindings.

Dumitru Ceara (3):
      ofctrl: Rename 'nb_cfg' to 'req_cfg'.
      controller: Implement a generic barrier based on ofctrl cur_cfg sync.
      binding: Set Logical_Switch_Port.up when all OVS flows are installed.


 NEWS                           |    6 +
 controller/automake.mk         |    2 
 controller/binding.c           |  181 ++++++++++++++++++++++++++++++
 controller/binding.h           |    5 +
 controller/ofctrl-seqno.c      |  245 ++++++++++++++++++++++++++++++++++++++++
 controller/ofctrl-seqno.h      |   50 ++++++++
 controller/ofctrl.c            |   70 ++++++-----
 controller/ofctrl.h            |    4 -
 controller/ovn-controller.c    |   53 +++++++--
 controller/test-ofctrl-seqno.c |  194 ++++++++++++++++++++++++++++++++
 northd/ovn-northd.c            |    4 -
 ovn-sb.ovsschema               |    5 -
 ovn-sb.xml                     |    8 +
 tests/automake.mk              |    8 +
 tests/ovn-ofctrl-seqno.at      |  226 +++++++++++++++++++++++++++++++++++++
 tests/ovn.at                   |   32 +++++
 tests/testsuite.at             |    1 
 utilities/ovn-sbctl.c          |    4 +
 18 files changed, 1046 insertions(+), 52 deletions(-)
 create mode 100644 controller/ofctrl-seqno.c
 create mode 100644 controller/ofctrl-seqno.h
 create mode 100644 controller/test-ofctrl-seqno.c
 create mode 100644 tests/ovn-ofctrl-seqno.at



More information about the dev mailing list