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

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

Reply via email to