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]>
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to