Thanks for the quick fix Dumitru, Acked-by: Mark Michelson <mmich...@redhat.com>
I went ahead and pushed this to main. On 4/26/22 11:35, Dumitru Ceara wrote:
When load balancers are configured there was a bug that was causing the default IN/OUT_ACL flows to be created with a wrong priority. This commit fixes it. Fixes: acec066091c8 ("northd: Add new NB_Global.options:default_acl_drop option.") Signed-off-by: Dumitru Ceara <dce...@redhat.com> --- northd/northd.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/northd/northd.c b/northd/northd.c index 56338a5b2..a56666297 100644 --- a/northd/northd.c +++ b/northd/northd.c @@ -6655,8 +6655,8 @@ build_acls(struct ovn_datapath *od, struct hmap *lflows, ovn_lflow_add(lflows, od, S_SWITCH_OUT_ACL, UINT16_MAX, "1", "next;"); } else { - ovn_lflow_add(lflows, od, S_SWITCH_IN_ACL, 1, "1", "next;"); - ovn_lflow_add(lflows, od, S_SWITCH_OUT_ACL, 1, "1", "next;"); + ovn_lflow_add(lflows, od, S_SWITCH_IN_ACL, 0, "1", "next;"); + ovn_lflow_add(lflows, od, S_SWITCH_OUT_ACL, 0, "1", "next;"); } ovn_lflow_add(lflows, od, S_SWITCH_IN_ACL_AFTER_LB, 0, "1", "next;"); } else {
_______________________________________________ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev