On 8/26/25 12:49 PM, Ales Musil wrote:
> On Tue, Aug 26, 2025 at 12:38 PM Ales Musil <[email protected]> wrote:
>
>> The SB MAC_Binding and Static_MAC_Binding use the same columns
>> as an indication of logical port and datapath. However, the check
>> if any of those rows is stale wasn't consistent. Unify it in a single
>> function to make sure that both tables have the same condition to
>> identify stale entries.
>>
>> Fixes: 417f1415b2f5 ("northd: Clean up SB MAC bindings for deleted ports.")
>> Fixes: 6856adc616a7 ("ovn-northd: Clear SB records depending on stale
>> datapaths.")
>>
>
> I forgot to add Reported-at: https://issues.redhat.com/browse/FDP-1623,
> I'll add it later in v2 or during marge depending on the reviews.
>
>
>> Signed-off-by: Ales Musil <[email protected]>
>> ---
Hi Ales,
Thanks for the fix! Aside from the typo in the title ("the the"), it
looks good to me. Assuming the ovsrobot CI run is successful:
Acked-by: Dumitru Ceara <[email protected]>
Regards,
Dumitru
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev