mbox series

[ovs-dev,v2,0/5] Fixed another set of flaky Unit Tests

Message ID 20231023092231.3861122-1-xsimonar@redhat.com
Headers show
Series Fixed another set of flaky Unit Tests | expand

Message

Xavier Simonart Oct. 23, 2023, 9:22 a.m. UTC 
Xavier Simonart (5):
  tests: fixed multiple tests not properly waiting for packets to be
    received
  tests: do not start backup-northd by default
  tests: fixed "LSP incremental processing"
  tests: fixed "ipsec -- basic configuration"
  tests: wait for all flows to be installed before sending packets

 tests/ovn-controller-vtep.at |   3 -
 tests/ovn-controller.at      |   6 +-
 tests/ovn-ipsec.at           |   4 +-
 tests/ovn-macros.at          |  10 +-
 tests/ovn-northd.at          |  42 ++---
 tests/ovn.at                 | 297 ++++++++++++-----------------------
 6 files changed, 129 insertions(+), 233 deletions(-)

Comments

Ales Musil Nov. 3, 2023, 9:23 a.m. UTC | #1 
On Mon, Oct 23, 2023 at 11:23 AM Xavier Simonart <xsimonar@redhat.com>
wrote:

> Xavier Simonart (5):
>   tests: fixed multiple tests not properly waiting for packets to be
>     received
>   tests: do not start backup-northd by default
>   tests: fixed "LSP incremental processing"
>   tests: fixed "ipsec -- basic configuration"
>   tests: wait for all flows to be installed before sending packets
>
>  tests/ovn-controller-vtep.at |   3 -
>  tests/ovn-controller.at      |   6 +-
>  tests/ovn-ipsec.at           |   4 +-
>  tests/ovn-macros.at          |  10 +-
>  tests/ovn-northd.at          |  42 ++---
>  tests/ovn.at                 | 297 ++++++++++++-----------------------
>  6 files changed, 129 insertions(+), 233 deletions(-)
>
> --
> 2.31.1
>
> _______________________________________________
> dev mailing list
> dev@openvswitch.org
> https://mail.openvswitch.org/mailman/listinfo/ovs-dev
>
>
The whole series looks good to me, thanks.

Acked-by: Ales Musil <amusil@redhat.com>
Dumitru Ceara Nov. 17, 2023, 2:25 p.m. UTC | #2 
On 11/3/23 10:23, Ales Musil wrote:
> On Mon, Oct 23, 2023 at 11:23 AM Xavier Simonart <xsimonar@redhat.com>
> wrote:
> 
>> Xavier Simonart (5):
>>   tests: fixed multiple tests not properly waiting for packets to be
>>     received
>>   tests: do not start backup-northd by default
>>   tests: fixed "LSP incremental processing"
>>   tests: fixed "ipsec -- basic configuration"
>>   tests: wait for all flows to be installed before sending packets
>>
>>  tests/ovn-controller-vtep.at |   3 -
>>  tests/ovn-controller.at      |   6 +-
>>  tests/ovn-ipsec.at           |   4 +-
>>  tests/ovn-macros.at          |  10 +-
>>  tests/ovn-northd.at          |  42 ++---
>>  tests/ovn.at                 | 297 ++++++++++++-----------------------
>>  6 files changed, 129 insertions(+), 233 deletions(-)
>>
>> --
>> 2.31.1
>>
>> _______________________________________________
>> dev mailing list
>> dev@openvswitch.org
>> https://mail.openvswitch.org/mailman/listinfo/ovs-dev
>>
>>
> The whole series looks good to me, thanks.
> 
> Acked-by: Ales Musil <amusil@redhat.com>
> 

Thanks, Xavier and Ales!

Applied to main and backported to all applicable stable branches down to
22.03.

Regards,
Dumitru