[ovs-dev] [PATCH ovn v2 2/2] ci: Enable AddressSanitizer in Linux clang CI test runs.

Dumitru Ceara dceara at redhat.com
Wed Dec 16 18:50:05 UTC 2020


On 12/16/20 7:29 PM, Mark Michelson wrote:
> On 12/16/20 1:12 PM, Ilya Maximets wrote:
>> On 12/16/20 1:59 PM, Dumitru Ceara wrote:
>>> To make sure no memory leaks or invalid accesses reported by
>>> AddressSanitizer are missed, also skip rechecking tests if
>>> AddressSanitizer reports are present in the test run directory.
>>>
>>> Note: This increases the GitHub Actions CI time for the clang test run
>>>        from ~9 minutes (before the change) to ~10 minutes (after the
>>>        change).
>>>
>>> Signed-off-by: Dumitru Ceara <dceara at redhat.com>
>>> ---
>>
>> Thanks, Dumitru!
>>
>> This change looks good to me.  I do not like that current OVN CI scripts
>> are using OVS_CFLAGS variable, but that is a different story.
>>
>> For this patch:
>> Acked-by: Ilya Maximets <i.maximets at ovn.org>
> 
> I pushed this series to master.
> 

Thanks!



More information about the dev mailing list