[ovs-dev,2/3] ovn-sb: Clarify ovn-northd's responsibilities.
diff mbox

Message ID 1441153303-86713-2-git-send-email-jpettit@nicira.com
State Accepted
Headers show

Commit Message

Justin Pettit Sept. 2, 2015, 12:21 a.m. UTC
The old documentation incorrectly attributed logical flow generation to
the CMS instead of ovn-northd.  This also corrects how a database is
referenced in the documentation.

Signed-off-by: Justin Pettit <jpettit@nicira.com>
---
 ovn/ovn-sb.xml |   23 ++++++++++++-----------
 1 files changed, 12 insertions(+), 11 deletions(-)

Comments

Russell Bryant Sept. 2, 2015, 12:34 a.m. UTC | #1
On 09/01/2015 08:21 PM, Justin Pettit wrote:
> The old documentation incorrectly attributed logical flow generation to
> the CMS instead of ovn-northd.  This also corrects how a database is
> referenced in the documentation.
> 
> Signed-off-by: Justin Pettit <jpettit@nicira.com>

Acked-by: Russell Bryant <rbryant@redhat.com>

Patch
diff mbox

diff --git a/ovn/ovn-sb.xml b/ovn/ovn-sb.xml
index 57e9689..38837c1 100644
--- a/ovn/ovn-sb.xml
+++ b/ovn/ovn-sb.xml
@@ -62,8 +62,8 @@ 
     The LN is a slave of the cloud management system running northbound of OVN.
     That CMS determines the entire OVN logical configuration and therefore the
     LN's content at any given time is a deterministic function of the CMS's
-    configuration, although that happens indirectly via the OVN Northbound DB
-    and <code>ovn-northd</code>.
+    configuration, although that happens indirectly via the
+    <ref db="OVN_Northbound"/> database and <code>ovn-northd</code>.
   </p>
 
   <p>
@@ -206,12 +206,13 @@ 
 
   <table name="Logical_Flow" title="Logical Network Flows">
     <p>
-      Each row in this table represents one logical flow.  The cloud management
-      system, via its OVN integration, populates this table with logical flows
-      that implement the L2 and L3 topology specified in the CMS configuration.
-      Each hypervisor, via <code>ovn-controller</code>, translates the logical
-      flows into OpenFlow flows specific to its hypervisor and installs them
-      into Open vSwitch.
+      Each row in this table represents one logical flow.
+      <code>ovn-northd</code> populates this table with logical flows
+      that implement the L2 and L3 topologies specified in the
+      <ref db="OVN_Northbound"/> database.  Each hypervisor, via
+      <code>ovn-controller</code>, translates the logical flows into
+      OpenFlow flows specific to its hypervisor and installs them into
+      Open vSwitch.
     </p>
 
     <p>
@@ -220,9 +221,9 @@ 
       flows are written in terms of logical ports and logical datapaths instead
       of physical ports and physical datapaths.  Translation between logical
       and physical flows helps to ensure isolation between logical datapaths.
-      (The logical flow abstraction also allows the CMS to do less work, since
-      it does not have to separately compute and push out physical flows to each
-      chassis.)
+      (The logical flow abstraction also allows the OVN centralized
+      components to do less work, since they do not have to separately
+      compute and push out physical flows to each chassis.)
     </p>
 
     <p>