This reverts commit 01277be6c833ef1d1c3fb568c139d3f0ac2b363a.

Commit [1] caused issues with handling ARP requests to vif ports inside
switch that has connectivity to the external network through a localnet port,
as well as many other problems, so it was suggested to revert the commit.
This commit was part of a fixes for this broken commit.

[1] 
https://github.com/ovn-org/ovn/commit/1b4058b9162c27edc6bd42c3dedbef622e2fd0a7
Reported-at: https://mail.openvswitch.org/pipermail/ovs-dev/2026-May/431983.html
Signed-off-by: Alexandra Rukomoinikova <[email protected]>
---
 northd/northd.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/northd/northd.c b/northd/northd.c
index 44f7db321..088033a43 100644
--- a/northd/northd.c
+++ b/northd/northd.c
@@ -10219,7 +10219,8 @@ build_lswitch_arp_chassis_resident(const struct 
ovn_datapath *od,
             op = node->data;
 
             ds_clear(&match);
-            ds_put_format(&match, "arp.op == 1 && inport == %s",
+            ds_put_format(&match, "(arp.op == 1 || arp.op == 2) "
+                                  "&& inport == %s",
                           op->json_key);
             ovn_lflow_add(lflows, od, S_SWITCH_IN_CHECK_PORT_SEC, 75,
                           ds_cstr(&match), REGBIT_EXT_ARP " = 1; next;",
-- 
2.48.1

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

Reply via email to