On 3/19/24 08:06, Ales Musil wrote:
> On Fri, Mar 15, 2024 at 7:43 PM Xavier Simonart <xsimo...@redhat.com> wrote:
>
>> Fixes: 65f9f010b426 ("tests: Check unit tests logs for errors.")
>> Signed-off-by: Xavier Simonart <xsimo...@redhat.com>
>> ---
>> tests/ovn-macros.at | 3 +++
>> 1 file changed, 3 insertions(+)
>>
>> diff --git a/tests/ovn-macros.at b/tests/ovn-macros.at
>> index 3410afb74..5c69facd2 100644
>> --- a/tests/ovn-macros.at
>> +++ b/tests/ovn-macros.at
>> @@ -48,12 +48,15 @@ m4_define([OVN_CLEANUP_SBOX],[
>> # "has no network name*" as localnet network_name is often set
>> "after" creating localnet.
>> # "receive tunnel port not found*" happening when closing other hv.
>> # "Failed to locate tunnel to reach main chassis" happening when
>> controller started e.g. before ovn-bridge-mappings is set
>> + # "Transaction causes multiple rows in \"MAC_Binding\" table to have
>> identical values"
>> + # happening as ovn-controller has a race condition over MAC binding
>> table with other controllers (GARP).
>> AT_CHECK([check_logs "
>> $error
>> /connection failed (No such file or directory)/d
>> /has no network name*/d
>> /receive tunnel port not found*/d
>> /Failed to locate tunnel to reach main chassis/d
>> + /Transaction causes multiple rows.*MAC_Binding/d
>> " $sbox])
>> ])
>>
>> --
>> 2.31.1
>>
>> _______________________________________________
>> dev mailing list
>> d...@openvswitch.org
>> https://mail.openvswitch.org/mailman/listinfo/ovs-dev
>>
>>
> Looks good to me, thanks.
>
> Acked-by: Ales Musil <amu...@redhat.com>
>
Thanks, Xavier and Ales!
Applied to main and 24.03.
Regards,
Dumitru
_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
