Thanks for the quick fix Dumitru,
Acked-by: Mark Michelson <[email protected]>
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 <[email protected]>
---
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
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
