diff mbox

[ovs-dev,1/3] ovn: Update ACL flow docs.

Message ID 1456955028-19535-2-git-send-email-russell@ovn.org
State Not Applicable
Headers show

Commit Message

Russell Bryant March 2, 2016, 9:43 p.m. UTC 
Apply some minor updates to the description of flows related to ACLs.

Signed-off-by: Russell Bryant <russell@ovn.org>
---
 ovn/northd/ovn-northd.8.xml |  8 +++++---
 ovn/ovn-sb.xml              | 11 +++++++++--
 2 files changed, 14 insertions(+), 5 deletions(-)
diff mbox

Patch

diff --git a/ovn/northd/ovn-northd.8.xml b/ovn/northd/ovn-northd.8.xml
index cacd760..902b916 100644
--- a/ovn/northd/ovn-northd.8.xml
+++ b/ovn/northd/ovn-northd.8.xml
@@ -157,9 +157,11 @@ 
       for the <code>from-lport</code> direction.  <code>allow</code>
       ACLs translate into logical flows with the <code>next;</code>
       action, <code>allow-related</code> ACLs translate into logical
-      flows with the <code>ct_next;</code> action, other ACLs translate
-      to <code>drop;</code>.  The <code>priority</code> values from the
-      <code>ACL</code> table are used directly.
+      flows with the <code>ct_commit; next;</code> actions, other ACLs
+      translate to <code>drop;</code>.  The <code>priority</code> values
+      from the <code>ACL</code> table have a limited range and have 1000
+      added to them to leave room for OVN default flows at both higher
+      and lower priorities.
     </p>
 
     <p>
diff --git a/ovn/ovn-sb.xml b/ovn/ovn-sb.xml
index f9ec691..476e870 100644
--- a/ovn/ovn-sb.xml
+++ b/ovn/ovn-sb.xml
@@ -905,8 +905,15 @@ 
 
         <dt><code>ct_commit;</code></dt>
         <dd>
-          Commit the flow to the connection tracking entry associated
-          with it by a previous call to <code>ct_next</code>.
+          <p>
+            Commit the flow to the connection tracking entry associated
+            with it by a previous call to <code>ct_next</code>.
+          </p>
+          <p>
+            Note that if you want processing to continue in the next table,
+            you must execute the <code>next</code> action after
+            <code>ct_commit</code>.
+          </p>
         </dd>
       </dl>