diff mbox

[ovs-dev] ovn-sb.xml: Remove obsolete documentation of OVN logical flows.

Message ID 1469858652-6657-1-git-send-email-blp@ovn.org
State Accepted
Headers show

Commit Message

Ben Pfaff July 30, 2016, 6:04 a.m. UTC
This description might have been accurate for 5 minutes or so, but no
longer than that.

Signed-off-by: Ben Pfaff <blp@ovn.org>
---
 ovn/ovn-sb.xml | 40 ++--------------------------------------
 1 file changed, 2 insertions(+), 38 deletions(-)

Comments

Ryan Moats July 30, 2016, 11:45 a.m. UTC | #1
"dev" <dev-bounces@openvswitch.org> wrote on 07/30/2016 01:04:12 AM:

> From: Ben Pfaff <blp@ovn.org>
> To: dev@openvswitch.org
> Cc: Ben Pfaff <blp@ovn.org>
> Date: 07/30/2016 01:04 AM
> Subject: [ovs-dev] [PATCH] ovn-sb.xml: Remove obsolete documentation
> of OVN logical flows.
> Sent by: "dev" <dev-bounces@openvswitch.org>
>
> This description might have been accurate for 5 minutes or so, but no
> longer than that.
>
> Signed-off-by: Ben Pfaff <blp@ovn.org>
> ---
>  ovn/ovn-sb.xml | 40 ++--------------------------------------
>  1 file changed, 2 insertions(+), 38 deletions(-)
>
> diff --git a/ovn/ovn-sb.xml b/ovn/ovn-sb.xml
> index c5f236e..694135f 100644
> --- a/ovn/ovn-sb.xml
> +++ b/ovn/ovn-sb.xml
> @@ -415,46 +415,10 @@
>      <p><em>Pipeline Stages</em></p>
>
>      <p>
> -      <code>ovn-northd</code> is responsible for populating the
> -      <ref table="Logical_Flow"/> table, so the stages are an
> -      implementation detail and subject to change.  This section
> -      describes the current logical flow table.
> +      <code>ovn-northd</code> populates the <ref table="Logical_Flow"/>
table
> +      with the logical flows described in detail in <code>ovn-
> northd</code>(8).
>      </p>
>
> -    <p>
> -      The ingress pipeline consists of the following stages:
> -    </p>
> -    <ul>
> -      <li>
> -        Port Security (Table 0): Validates the source address, drops
> -        packets with a VLAN tag, and, if configured, verifies that the
> -        logical port is allowed to send with the source address.
> -      </li>
> -
> -      <li>
> -        L2 Destination Lookup (Table 1): Forwards known unicast
> -        addresses to the appropriate logical port.  Unicast packets to
> -        unknown hosts are forwarded to logical ports configured with the
> -        special <code>unknown</code> mac address.  Broadcast, and
> -        multicast are flooded to all ports in the logical switch.
> -      </li>
> -    </ul>
> -
> -    <p>
> -      The egress pipeline consists of the following stages:
> -    </p>
> -    <ul>
> -      <li>
> -        ACL (Table 0): Applies any specified access control lists.
> -      </li>
> -
> -      <li>
> -        Port Security (Table 1): If configured, verifies that the
> -        logical port is allowed to receive packets with the destination
> -        address.
> -      </li>
> -    </ul>
> -
>      <column name="logical_datapath">
>        The logical datapath to which the logical flow belongs.
>      </column>
> --

Agreed, circular file it...

Acked-by: Ryan Moats <rmoats@us.ibm.com>
Ben Pfaff July 30, 2016, 5:34 p.m. UTC | #2
On Sat, Jul 30, 2016 at 06:45:50AM -0500, Ryan Moats wrote:
> "dev" <dev-bounces@openvswitch.org> wrote on 07/30/2016 01:04:12 AM:
> 
> > From: Ben Pfaff <blp@ovn.org>
> > To: dev@openvswitch.org
> > Cc: Ben Pfaff <blp@ovn.org>
> > Date: 07/30/2016 01:04 AM
> > Subject: [ovs-dev] [PATCH] ovn-sb.xml: Remove obsolete documentation
> > of OVN logical flows.
> > Sent by: "dev" <dev-bounces@openvswitch.org>
> >
> > This description might have been accurate for 5 minutes or so, but no
> > longer than that.
> >
> > Signed-off-by: Ben Pfaff <blp@ovn.org>
> 
> Agreed, circular file it...
> 
> Acked-by: Ryan Moats <rmoats@us.ibm.com>

Thanks, applied.
diff mbox

Patch

diff --git a/ovn/ovn-sb.xml b/ovn/ovn-sb.xml
index c5f236e..694135f 100644
--- a/ovn/ovn-sb.xml
+++ b/ovn/ovn-sb.xml
@@ -415,46 +415,10 @@ 
     <p><em>Pipeline Stages</em></p>
 
     <p>
-      <code>ovn-northd</code> is responsible for populating the
-      <ref table="Logical_Flow"/> table, so the stages are an
-      implementation detail and subject to change.  This section
-      describes the current logical flow table.
+      <code>ovn-northd</code> populates the <ref table="Logical_Flow"/> table
+      with the logical flows described in detail in <code>ovn-northd</code>(8).
     </p>
 
-    <p>
-      The ingress pipeline consists of the following stages:
-    </p>
-    <ul>
-      <li>
-        Port Security (Table 0): Validates the source address, drops
-        packets with a VLAN tag, and, if configured, verifies that the
-        logical port is allowed to send with the source address.
-      </li>
-
-      <li>
-        L2 Destination Lookup (Table 1): Forwards known unicast
-        addresses to the appropriate logical port.  Unicast packets to
-        unknown hosts are forwarded to logical ports configured with the
-        special <code>unknown</code> mac address.  Broadcast, and
-        multicast are flooded to all ports in the logical switch.
-      </li>
-    </ul>
-
-    <p>
-      The egress pipeline consists of the following stages:
-    </p>
-    <ul>
-      <li>
-        ACL (Table 0): Applies any specified access control lists.
-      </li>
-
-      <li>
-        Port Security (Table 1): If configured, verifies that the
-        logical port is allowed to receive packets with the destination
-        address.
-      </li>
-    </ul>
-
     <column name="logical_datapath">
       The logical datapath to which the logical flow belongs.
     </column>