Remove 'ip4' from the match condition for this flow.

Signed-off-by: Han Zhou <hz...@ovn.org>
---
 northd/northd.c     | 2 +-
 tests/ovn-northd.at | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/northd/northd.c b/northd/northd.c
index da3c8cf770bc..c4cb7232e0a1 100644
--- a/northd/northd.c
+++ b/northd/northd.c
@@ -13109,7 +13109,7 @@ build_misc_local_traffic_drop_flows_for_lrouter(
 
     /* TTL discard */
     ovn_lflow_add(lflows, od, S_ROUTER_IN_IP_INPUT, 30,
-                  "ip4 && ip.ttl == {0, 1}", debug_drop_action());
+                  "ip.ttl == {0, 1}", debug_drop_action());
 
     /* Pass other traffic not already handled to the next table for
      * routing. */
diff --git a/tests/ovn-northd.at b/tests/ovn-northd.at
index 9fee00094419..f0df5a284c1a 100644
--- a/tests/ovn-northd.at
+++ b/tests/ovn-northd.at
@@ -6785,7 +6785,7 @@ AT_CHECK([grep -e 'lr_in_ip_input    ' lrflows | grep -e 
'igmp' -e 'mld' -e 'ip.
   table=??(lr_in_ip_input     ), priority=100  , match=(inport == "lrp1" && 
ip6 && ip6.src == 1010::/64 && ip.ttl == {0, 1} && !ip.later_frag), 
action=(icmp6 {eth.dst <-> eth.src; ip6.dst = ip6.src; ip6.src = 1010::1 ; 
ip.ttl = 254; icmp6.type = 3; /* Time exceeded */ icmp6.code = 0; /* TTL 
exceeded in transit */ outport = "lrp1"; flags.loopback = 1; output; };)
   table=??(lr_in_ip_input     ), priority=100  , match=(inport == "lrp2" && 
ip4 && ip.ttl == {0, 1} && !ip.later_frag), action=(icmp4 {eth.dst <-> eth.src; 
icmp4.type = 11; /* Time exceeded */ icmp4.code = 0; /* TTL exceeded in transit 
*/ ip4.dst = ip4.src; ip4.src = 20.20.20.1 ; ip.ttl = 254; outport = "lrp2"; 
flags.loopback = 1; output; };)
   table=??(lr_in_ip_input     ), priority=100  , match=(inport == "lrp2" && 
ip6 && ip6.src == 2020::/64 && ip.ttl == {0, 1} && !ip.later_frag), 
action=(icmp6 {eth.dst <-> eth.src; ip6.dst = ip6.src; ip6.src = 2020::1 ; 
ip.ttl = 254; icmp6.type = 3; /* Time exceeded */ icmp6.code = 0; /* TTL 
exceeded in transit */ outport = "lrp2"; flags.loopback = 1; output; };)
-  table=??(lr_in_ip_input     ), priority=30   , match=(ip4 && ip.ttl == {0, 
1}), action=(drop;)
+  table=??(lr_in_ip_input     ), priority=30   , match=(ip.ttl == {0, 1}), 
action=(drop;)
 ])
 
 dnl Flows to "route" (statically forward) without decrementing TTL for
-- 
2.30.2

_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to