On 28 Feb 2023, at 17:23, Simon Horman wrote:

> On Tue, Feb 28, 2023 at 04:24:57PM +0100, Eelco Chaudron wrote:
>>
>>
>> On 16 Feb 2023, at 13:21, Eelco Chaudron wrote:
>>
>>> On 15 Feb 2023, at 21:15, Ilya Maximets wrote:
>>>
>>>> On 2/14/23 14:54, Eelco Chaudron wrote:
>>>>> Run "make check-offloads" as part of the GitHub actions tests.
>>>>>
>>>>> This test was run 25 times using GitHub actions, and the
>>>>> failing rerun test cases where excluded. There are quite some
>>>>> first-run failures, but unfortunately, there is no other
>>>>> more stable kernel available as a GitHub-hosted runner.
>>>>>
>>>>> Signed-off-by: Eelco Chaudron <[email protected]>
>>>>> ---
>>>>
>>>> It doesn't seems to take that much time.  Maybe it's not that
>>>> bad of an idea after all. :)
>>>>
>>>> It might make sense to run this job with asan/ubsan enabled.
>>>> What do you think?  We might re-structure asan/ubsan jobs to
>>>> run them at the same time for this, like OVN does.
>>>
>>> I think this was giving a lot of errors, but they might all be fixed by 
>>> Mike’s fixes.
>>>
>>> I’ll take a look and see what OVN does…
>>
>> I did make the change to combine the UBSAN and ASAN tests, but I did not 
>> enable it for this specific test.
>>
>> The tests takes too long (more than 30 min, so it’s canceled by GitHub), and 
>> there are some failures. Quickly looking at some it seems timing-related.
>
> FWIIW, OVN seems to take the approach of splitting system tests into three
> batches: 1-99, 100-199, 200-. Or something like that.

Thanks for bringing this up. I’ll put a note in my to-do list and will try to 
work on this later, and see if I can fix/exclude some of the timing issues.

//Eelco

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

Reply via email to