On 1/25/21 12:02 PM, Numan Siddique wrote:
On Fri, Jan 22, 2021 at 5:01 PM Dumitru Ceara <[email protected] <mailto:[email protected]>> wrote:The ACLs configured by the "ovn -- L2 Drop and Allow ACL w/ Stateful ACL" test were incorrect because they didn't enclose logical port names in quotes. This caused stateful ACLs to never be applied so the L2 drop rules were not properly tested. ovn-controller was logging the following errors: lflow|WARN|error parsing match "reg0[8] == 1 && (inport == lp31 && ip)": Syntax error at `lp31' expecting constant. lflow|WARN|error parsing match "reg0[8] == 1 && (inport == lp31 && ip)": Syntax error at `lp31' expecting constant. lflow|WARN|error parsing match "reg0[8] == 1 && (inport == lp31 && ip)": Syntax error at `lp31' expecting constant. Fixes: 63640c0d1199 ("ovn-northd: ls_*_acl behavior not consistent for untracked flows") Signed-off-by: Dumitru Ceara <[email protected] <mailto:[email protected]>>Good catch. Thanks. I applied this patch to master and backported upto 20.06 branch.
Thanks! _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
