On Tue, Aug 16, 2022 at 1:26 PM Dumitru Ceara <[email protected]> wrote:

> On 8/16/22 13:05, Dumitru Ceara wrote:
> > On 8/16/22 08:33, Ales Musil wrote:
> >> Add MAC binding aging mechanism, that utilizes
> >> the timestamp column of MAC_Binding table.
> >> When the MAC binding exceeds the threshold it is
> >> removed from SB DB, this is postponed only in case
> >> we receive update ARP with update to MAC address.
> >>
> >> The threshold is configurable via option
> >> "mac_binding_age_threshold" that can be specified
> >> for each logical router. The option is defaulting to
> >> 0 which means that by default the aging is disabled
> >> and the MAC binding rows will be persisted the same
> >> way as before.
> >>
> >> Reported-at: https://bugzilla.redhat.com/2084668
> >> Acked-by: Mark Michelson <[email protected]>
> >> Acked-by: Han Zhou <[email protected]>
> >> Signed-off-by: Ales Musil <[email protected]>
> >> ---
> >> v4: Rebase on top of current main.
> >>     Skip one row instead of all if time elapsed is negative.
> >>     Add note to NEWS.
> >> v5: Rebase on top of current main.
> >>     Address comments from Han, Numan and Dumitru.
> >> v6: Rebase on top of current main.
> >> ---
> >
> > Acked-by: Dumitru Ceara <[email protected]>
> >
>
> Actually, I have to condition this ack by the following incremental
> change (the new test is racy otherwise).
>
> Thanks!
>
> diff --git a/tests/ovn.at b/tests/ovn.at
> index 11094eff5..27dc11498 100644
> --- a/tests/ovn.at
> +++ b/tests/ovn.at
> @@ -32474,6 +32474,8 @@ ovs-vsctl -- add-port br-phys ext2 -- \
>  ovs-vsctl set open . external_ids:ovn-bridge-mappings=physnet1:br-phys
>
>  OVN_POPULATE_ARP
> +wait_for_ports_up
> +check ovn-nbctl --wait=hv sync
>
>  send_garp() {
>      hv=$1
>
>
That one slipped through, thank you.

Regards,
Ales

-- 

Ales Musil

Senior Software Engineer - OVN Core

Red Hat EMEA <https://www.redhat.com>

[email protected]    IM: amusil
<https://red.ht/sig>
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to