diff mbox

[ovs-dev,RFC,1/5] OVN: SFC Implemnentation - documentation for SFC ACL Action

Message ID ac43c306cc25626ff65bc9eb8201d961608eb6c8.1482874169.git.jmcdowall@paloaltonetworks.com
State RFC
Headers show

Commit Message

John McDowall Dec. 27, 2016, 10:11 p.m. UTC
This is minor changes to documentation to support additional ACL action.

Co-authored-by: Flavio Fernandes <flavio at flaviof.com>
Reported at: https://mail.openvswitch.org/pipermail/ovs-discuss/2016-March/040381.html
Reported at: https://mail.openvswitch.org/pipermail/ovs-discuss/2016-May/041359.html

Signed-off-by: John McDowall <jmcdowall@paloaltonetworks.com>

---
 ovn/northd/ovn-northd.8.xml | 5 +++++
 1 file changed, 5 insertions(+)

Comments

Ben Pfaff Jan. 31, 2017, 9:42 p.m. UTC | #1
On Tue, Dec 27, 2016 at 02:11:42PM -0800, John McDowall wrote:
> This is minor changes to documentation to support additional ACL action.
> 
> Co-authored-by: Flavio Fernandes <flavio at flaviof.com>
> Reported at: https://mail.openvswitch.org/pipermail/ovs-discuss/2016-March/040381.html
> Reported at: https://mail.openvswitch.org/pipermail/ovs-discuss/2016-May/041359.html
> 
> Signed-off-by: John McDowall <jmcdowall@paloaltonetworks.com>

This is documentation for something coming later in the series, right?
Usually we document a feature in the same patch that implements it.
diff mbox

Patch

diff --git a/ovn/northd/ovn-northd.8.xml b/ovn/northd/ovn-northd.8.xml
index f3c1682d1..380a53455 100644
--- a/ovn/northd/ovn-northd.8.xml
+++ b/ovn/northd/ovn-northd.8.xml
@@ -304,6 +304,11 @@ 
         connections.
       </li>
       <li>
+        <code>sfc</code> ACLs work as entry points for service function
+        chaining, also known as SFC classifiers. Further attributes such
+        as what chain to be used are provided via the options column.
+      </li>
+      <li>
         Other ACLs translate to <code>drop;</code> for new or untracked
         connections and <code>ct_commit(ct_label=1/1);</code> for known
         connections.  Setting <code>ct_label</code> marks a connection