On 10 March 2017 at 16:10, Jarno Rajahalme <[email protected]> wrote:
> Normally ICMPP responses are in the reply direction of a conntrack

's/ICMPP/ICMP/'

> entry.  This test exercises an ICMP response to the original direction
> of the conntrack entry.
>
> Signed-off-by: Jarno Rajahalme <[email protected]>
> ---

Somehow this slipped past my radar. Might want to roll in the
incremental I posted below, but otherwise LGTM.

Acked-by: Joe Stringer <[email protected]>

diff --git a/tests/system-traffic.at b/tests/system-traffic.at
index 62eb7bda7e31..f66ed10501a5 100644
--- a/tests/system-traffic.at
+++ b/tests/system-traffic.at
@@ -1392,7 +1392,7 @@ priority=1,action=drop
table=1,ip,action=ct(zone=34673,table=2)
table=2,in_port=2,udp,action=ct(commit,zone=34673),1
table=2,in_port=1,udp,action=ct(commit,zone=34673),2
-table=2,in_port=2,icmp,action=1
+table=2,in_port=2,ct_state=+rel,icmp,action=1
])

AT_CHECK([ovs-ofctl --bundle add-flows br0 flows.txt])
@@ -1421,7 +1421,7 @@ AT_CHECK([ovs-ofctl dump-flows br0 | ofctl_strip
| sort | grep -v drop], [0], [d
 table=1, n_packets=4, n_bytes=224, ip actions=ct(table=2,zone=34673)
 table=2, n_packets=1, n_bytes=42, udp,in_port=1
actions=ct(commit,zone=34673),output:2
 table=2, n_packets=1, n_bytes=42, udp,in_port=2
actions=ct(commit,zone=34673),output:1
- table=2, n_packets=2, n_bytes=140, icmp,in_port=2 actions=output:1
+ table=2, n_packets=2, n_bytes=140, ct_state=+rel,icmp,in_port=2
actions=output:1
NXST_FLOW reply:
])
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to