[ovs-dev] [PATCH 0/2] ovn: Make ovn-controller readonly to the Chassis and Port_Binding SB DB tables

Numan Siddique nusiddiq at redhat.com
Fri Nov 11 15:22:57 UTC 2016


With the commit - "ovsdb-server: Implement read-only remote connection type." merged, this patch series is the next step in making ovn-controller a read-only client to the Southbound DB.

Patch 1 removes the chassis creation code from ovn-controller. It now expects the chassis row to be created in the Chassis table.
Patch 2 removes the ovn-controller setting the chassis column of Port binding. Instead ovn-northd will set it.

Since ovn-controller is not setting the chassis column, one side effect of this patchset is the Logical_Switch_Port.up doesn't reflect the
true value. Is this limitation acceptable ?



Numan Siddique (2):
  ovn-controller: Remove chassis creation code from ovn-controller
  ovn: Move setting of chassis column of port binding to ovn-northd

 ovn/controller/binding.c            |  86 +++-------------
 ovn/controller/chassis.c            | 195 +++++++++---------------------------
 ovn/controller/ovn-controller.8.xml |  60 -----------
 ovn/controller/physical.c           |   7 ++
 ovn/northd/ovn-northd.c             |  33 +++++-
 ovn/ovn-architecture.7.xml          |  28 ++----
 ovn/ovn-nb.xml                      |   8 ++
 ovn/ovn-sb.xml                      | 116 ++++++++++-----------
 ovn/utilities/ovn-sbctl.c           |  15 +--
 tests/ofproto-macros.at             |   1 +
 tests/ovn-controller.at             |  37 +++----
 tests/ovn.at                        |  84 ++++++++++++++--
 tutorial/ovs-sandbox                |   1 +
 13 files changed, 276 insertions(+), 395 deletions(-)

-- 
2.7.4



More information about the dev mailing list