On 3/18/25 2:56 PM, Ales Musil wrote: > The test would fail from time to time on transaction error when > two controllers try to add the same FDB into DB simultaneously. > Ignore the transaction error during cleanup. > > Signed-off-by: Ales Musil <[email protected]> > ---
Hi Ales, Thanks for the fix! > tests/ovn.at | 11 +++++++++-- > 1 file changed, 9 insertions(+), 2 deletions(-) > > diff --git a/tests/ovn.at b/tests/ovn.at > index 714286596..d7ae18e2f 100644 > --- a/tests/ovn.at > +++ b/tests/ovn.at > @@ -5937,8 +5937,15 @@ for i in 1 2 3; do > done > done > > -OVN_CLEANUP([hv1],[hv2],[hv3]) > - > +# There might be transaction error for FDB when two controllers try to add > +# the same FDB simultaneously. > +OVN_CLEANUP([hv1 > +/transaction error/d > +],[hv2 > +/transaction error/d > +],[hv3 > +/transaction error/d > +]) > AT_CLEANUP > ]) > However, for MAC_Bindings we ignore such logs for all tests, in OVN_CLEANUP_SBOX(). Should we do the same for FDB? Also, "transaction error" is a bit too generic. It might be good to match on the specific table name. What do you think? Regards, Dumitru _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
