Re: [ovs-dev] [PATCH ovn 2/2] ci: Remove the ASAN ARM64 ASAN workaround

2023-10-16 Thread Ales Musil
On Tue, Oct 17, 2023 at 7:08 AM Ales Musil wrote: > On Mon, Oct 16, 2023 at 10:20 PM Mark Michelson > wrote: > > > > Hi Ales, > > > > The patch makes sense to me. Is there something that guarantees that > > clang >= 16 is installed when running CI? > > Yeah, clang 16 is default on Fedora 38 whic

Re: [ovs-dev] [PATCH ovn 2/2] ci: Remove the ASAN ARM64 ASAN workaround

2023-10-16 Thread Ales Musil
On Mon, Oct 16, 2023 at 10:20 PM Mark Michelson wrote: > > Hi Ales, > > The patch makes sense to me. Is there something that guarantees that > clang >= 16 is installed when running CI? Yeah, clang 16 is default on Fedora 38 which we use for ARM64 runs. > > Thanks, > Mark Michelson > > On 10/16/2

Re: [ovs-dev] [PATCH ovn 2/2] ci: Remove the ASAN ARM64 ASAN workaround

2023-10-16 Thread Mark Michelson
Hi Ales, The patch makes sense to me. Is there something that guarantees that clang >= 16 is installed when running CI? Thanks, Mark Michelson On 10/16/23 04:08, Ales Musil wrote: The clang from version 16 and further fixes the issue which was causing the slowness. Remove the workaround whic

[ovs-dev] [PATCH ovn 2/2] ci: Remove the ASAN ARM64 ASAN workaround

2023-10-16 Thread Ales Musil
The clang from version 16 and further fixes the issue which was causing the slowness. Remove the workaround which allows the leak sanitizers to run on ARM64 as well. Signed-off-by: Ales Musil --- .ci/ci.sh | 5 - 1 file changed, 5 deletions(-) diff --git a/.ci/ci.sh b/.ci/ci.sh index 6bb211