On 5/2/23 12:08, Dumitru Ceara wrote:
On 4/21/23 21:51, Mark Michelson wrote:
This allows for evaluating ACLs at the current tier to stop, and to
start evaluating ACLs at the next tier. If not using tiers, or if we
match on the final ACL tier, then a "pass" verdict results in the
default ACL action being applied.

Reported-at: https://bugzilla.redhat.com/show_bug.cgi?id=2134138

Signed-off-by: Mark Michelson <[email protected]>
---
  northd/northd.c       |  8 +++++++-
  ovn-nb.ovsschema      |  4 ++--
  ovn-nb.xml            | 10 ++++++++++
  tests/ovn-northd.at   | 46 +++++++++++++++++++++++++++++++++++++++++++
  tests/system-ovn.at   | 41 ++++++++++++++++++++++++++++++++++----
  utilities/ovn-nbctl.c |  2 +-
  6 files changed, 103 insertions(+), 8 deletions(-)


Hi Mark,

I didn't review the series closely but one thing we should add is a NEWS
entry for the new verdict and for the multi-tier support.

Yes, this is good. In v4, I add the NEWS entry to patch 3, since patch 3 is where the tiered ACL support is initially added.


Regards,
Dumitru


_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to