On Fri, Jul 24, 2026 at 1:16 PM Ilya Maximets <[email protected]> wrote:
> On 7/22/26 10:18 AM, Ales Musil via dev wrote: > > We already have the MAC address within the MAC binding row. There > > is no need to flood the ARP request as this serves as liveness > > check. Send the ARP probe as unicast instead. > > > > Signed-off-by: Ales Musil <[email protected]> > > --- > > [...] > > > @@ -37538,7 +37538,8 @@ send_imcp_echo_req hv1 public 00:00:00:00:10:00 > 00:00:00:00:10:1a 192.168.20.2 1 > > OVS_WAIT_UNTIL([test $(grep -c "Sending ARP/ND.*ip: 192.168.20.2" > hv1/ovn-controller.log) -eq 2]) > > > > check ovn-nbctl --wait=hv acl-del join > > -wait_row_count mac_binding 0 > > +wait_row_count mac_binding 0 ip="192.168.10.100" > logical_port="gw-public" > > +wait_row_count mac_binding 0 ip="192.168.20.2" logical_port="gw-join" > > > > send_icmp6_echo_req hv1 public 00:00:00:00:10:00 00:00:00:00:50:01 > fd12::2 fd11::64 > > wait_row_count mac_binding 1 mac=\"00:00:00:00:30:00\" ip=\"fd12::2\" > > @@ -37555,7 +37556,7 @@ sleep $((aging_th / 2)) > > send_icmp6_echo_req hv1 public 00:00:00:00:10:00 00:00:00:00:50:01 > fd12::2 fd11::64 > > OVS_WAIT_UNTIL([test $(grep -c "Sending ARP/ND.*ip: fd12::2" > hv1/ovn-controller.log) -eq 2]) > > > > -wait_row_count mac_binding 0 > > +wait_row_count mac_binding 0 ip=\"fd12::2\" logical_port="gw-join" > Commit message should mention why these checks have been narrowed. > It is not obvious. Otherwise, LGTM. > > Acked-by: Ilya Maximets <[email protected]> > > Thank you Ilya, that makes sense, I have updated the commit message in v2. Regards, Ales _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
