[ovs-git] [ovn-org/ovn] daeaa2: tests: Fix L2 ACL test.

Dumitru Ceara noreply at github.com
Mon Jan 25 10:57:06 UTC 2021


  Branch: refs/heads/branch-20.12
  Home:   https://github.com/ovn-org/ovn
  Commit: daeaa2b255442a6e6e63631c4e4e851673d57e44
      https://github.com/ovn-org/ovn/commit/daeaa2b255442a6e6e63631c4e4e851673d57e44
  Author: Dumitru Ceara <dceara at redhat.com>
  Date:   2021-01-25 (Mon, 25 Jan 2021)

  Changed paths:
    M tests/ovn.at

  Log Message:
  -----------
  tests: Fix L2 ACL test.

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 <dceara at redhat.com>
Signed-off-by: Numan Siddique <numans at ovn.org>

(cherry-picked from master commit a5ac78e18bc47f6f695f595ebbe6d639c6f9a48e)




More information about the git mailing list