Thanks for fixing this Ales,
Acked-by: Mark Michelson <mmich...@redhat.com>
I went ahead and applied this to main.
On 5/19/23 04:22, Ales Musil wrote:
The test was failing on Fedora with "grep: warning: stray \ before ,"
due to autotest stripping the "[" and "]". Use the check
directly instead.
Fixes: 46dacbb755dd ("northd: Break ACLs into two stages.")
Signed-off-by: Ales Musil <amu...@redhat.com>
---
tests/ovn.at | 7 ++++---
1 file changed, 4 insertions(+), 3 deletions(-)
diff --git a/tests/ovn.at b/tests/ovn.at
index 9e6e8a14a..6f9fbbfd2 100644
--- a/tests/ovn.at
+++ b/tests/ovn.at
@@ -32018,11 +32018,12 @@ check ovn-nbctl --wait=hv sync
dp_key=$(fetch_column Datapath_Binding tunnel_key external_ids:name=ls)
rtr_port_key=$(fetch_column Port_Binding tunnel_key logical_port=ls_lr)
-ovs-ofctl dump-flows br-int table=16 | grep "reg14=0x${rtr_port_key},metadata=0x${dp_key},nw_dst=42.42.42.42 actions=load:0x1->OXM_OF_PKT_REG4[49],resubmit(,17)"
+ovs-ofctl dump-flows br-int table=16 | grep
"reg14=0x${rtr_port_key},metadata=0x${dp_key},nw_dst=42.42.42.42"
# Check that ovn-controller adds a flow to drop packets with dest IP
# 42.42.42.42 coming from the router port.
-AT_CHECK([ovs-ofctl dump-flows br-int table=16 | grep
"reg14=0x${rtr_port_key},metadata=0x${dp_key},nw_dst=42.42.42.42
actions=load:0x1->OXM_OF_PKT_REG4\[\[49\]\],resubmit(,17)" -c], [0], [dnl
-1
+AT_CHECK_UNQUOTED([ovs-ofctl dump-flows br-int table=16 | ofctl_strip_all | \
+ grep "reg14=0x${rtr_port_key},metadata=0x${dp_key},nw_dst=42.42.42.42"],
[0], [dnl
+ table=16,
priority=1001,ip,reg14=0x${rtr_port_key},metadata=0x${dp_key},nw_dst=42.42.42.42
actions=load:0x1->OXM_OF_PKT_REG4[[49]],resubmit(,17)
])
OVN_CLEANUP([hv1])
_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev