the documentation for tcp_reset{} does not include nested actions.
Someone following the doc prior to this patch would get a parse error.
Add the required {} for tcp_reset.

Fixes: 3c9fa1172400 ("OVN: add tcp_reset{} action support")
Assisted-by: Claude Opus 4.6, Claude Code
Signed-off-by: Jacob Tanenbaum <[email protected]>
---
 ovn-sb.xml | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/ovn-sb.xml b/ovn-sb.xml
index 02ffb6e44..73816098e 100644
--- a/ovn-sb.xml
+++ b/ovn-sb.xml
@@ -2510,7 +2510,7 @@
           <p><b>Prerequisite:</b> <code>ip6</code></p>
         </dd>
 
-        <dt><code>tcp_reset;</code></dt>
+        <dt><code>tcp_reset { <var>action</var>; </code>...<code> 
};</code></dt>
         <dd>
           <p>
             This action transforms the current TCP packet according to the
@@ -2530,6 +2530,7 @@ tcp.flags = RST;
           <p>
             Then, the action drops all TCP options and payload data, and
             updates the TCP checksum. IP ttl is set to 255.
+            The nested actions are executed on the resulting RST packet.
           </p>
 
           <p><b>Prerequisite:</b> <code>tcp</code></p>
-- 
2.55.0

_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to