Ports of type 'localport' are present on every hypervisor and ovn-controller is sending gARPs for them which makes upstream switches to see its MAC address flapping.
In order to avoid this behavior, the current patch is skipping localports when sending gARP/RARP packets. Reported-at: https://bugzilla.redhat.com/show_bug.cgi?id=1939470 Signed-off-by: Daniel Alvarez Sanchez <dalva...@redhat.com> Signed-off-by: Dumitru Ceara <dce...@redhat.com> --- v1 -> v2: added test controller/pinctrl.c | 6 +++++ tests/ovn.at | 55 ++++++++++++++++++++++++++++++++++++++++++++ 2 files changed, 61 insertions(+) diff --git a/controller/pinctrl.c b/controller/pinctrl.c index b42288ea5..523a45b9a 100644 --- a/controller/pinctrl.c +++ b/controller/pinctrl.c @@ -4240,6 +4240,12 @@ send_garp_rarp_update(struct ovsdb_idl_txn *ovnsb_idl_txn, struct shash *nat_addresses) { volatile struct garp_rarp_data *garp_rarp = NULL; + + /* Skip localports as they don't need to be announced */ + if (!strcmp(binding_rec->type, "localport")) { + return; + } + /* Update GARP for NAT IP if it exists. Consider port bindings with type * "l3gateway" for logical switch ports attached to gateway routers, and * port bindings with type "patch" for logical switch ports attached to diff --git a/tests/ovn.at b/tests/ovn.at index b751d6db2..8c0146c2d 100644 --- a/tests/ovn.at +++ b/tests/ovn.at @@ -11596,6 +11596,61 @@ OVN_CLEANUP([hv1],[hv2]) AT_CLEANUP ]) +OVN_FOR_EACH_NORTHD([ +AT_SETUP([ovn -- localport suppress gARP]) +ovn_start + +net_add n1 +sim_add hv1 +as hv1 +check ovs-vsctl add-br br-phys +ovn_attach n1 br-phys 192.168.0.1 + +check ovs-vsctl set open . external-ids:ovn-bridge-mappings=phys:br-phys + +check ovn-nbctl ls-add ls \ + -- lsp-add ls lp \ + -- lsp-set-type lp localport \ + -- lsp-set-addresses lp "00:00:00:00:00:01 10.0.0.1" \ + -- lsp-add ls ln \ + -- lsp-set-type ln localnet \ + -- lsp-set-options ln network_name=phys \ + -- lsp-add ls lsp \ + -- lsp-set-addresses lsp "00:00:00:00:00:02 10.0.0.2" + +dnl First bind the localport. +check ovs-vsctl add-port br-int vif1 \ + -- set Interface vif1 external-ids:iface-id=lp +check ovn-nbctl --wait=hv sync + +dnl Then bind the regular vif. +check ovs-vsctl add-port br-int vif2 \ + -- set Interface vif2 external-ids:iface-id=lsp \ + options:tx_pcap=hv1/vif2-tx.pcap \ + options:rxq_pcap=hv1/vif2-rx.pcap + +wait_for_ports_up lsp +check ovn-nbctl --wait=hv sync + +dnl Wait for at least two gARPs from lsp (10.0.0.2). +lsp_garp=ffffffffffff000000000002080600010800060400010000000000020a0000020000000000000a000002 +OVS_WAIT_UNTIL([ + garps=`$PYTHON "$ovs_srcdir/utilities/ovs-pcap.in" hv1/br-phys-tx.pcap | grep ${lsp_garp} -c` + test $garps -ge 2 +]) + +dnl At this point it's safe to assume that ovn-controller skipped sending gARP +dnl for the localport. Check that there are no other packets than the gARPs +dnl for the regular vif. +AT_CHECK([ + pkts=`$PYTHON "$ovs_srcdir/utilities/ovs-pcap.in" hv1/br-phys-tx.pcap | grep -v ${lsp_garp} -c` + test 0 -eq $pkts +]) + +OVN_CLEANUP([hv1]) +AT_CLEANUP +]) + OVN_FOR_EACH_NORTHD([ AT_SETUP([ovn -- 1 LR with HA distributed router gateway port]) ovn_start -- _______________________________________________ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev
