Skip to content

Commit

Permalink
ci: Apply the ASAN workaround only for Clang <16
Browse files Browse the repository at this point in the history
The clang from version 16 and further fixes
the issue which was causing the slowness.
Remove the workaround for version with the fix
applied which allows the leak sanitizers to
run on ARM64 as well.

Signed-off-by: Ales Musil <amusil@redhat.com>
Acked-by: Simon Horman <horms@ovn.org>
Acked-by: Mark Michelson <mmichels@redhat.com>
Signed-off-by: Mark Michelson <mmichels@redhat.com>
  • Loading branch information
almusil authored and putnopvut committed Oct 18, 2023
1 parent c013529 commit d19b438
Showing 1 changed file with 11 additions and 1 deletion.
12 changes: 11 additions & 1 deletion .ci/ci.sh
Original file line number Diff line number Diff line change
Expand Up @@ -105,6 +105,16 @@ function run_tests() {
"
}

function check_clang_version_ge() {
lower=$1
version=$(clang --version | head -n1 | cut -d' ' -f3)
if ! echo -e "$lower\n$version" | sort -CV; then
return 1
fi

return 0
}

options=$(getopt --options "" \
--long help,shell,archive-logs,jobs:,ovn-path:,ovs-path:,image-name:\
-- "${@}")
Expand Down Expand Up @@ -149,7 +159,7 @@ while true; do
done

# Workaround for https://bugzilla.redhat.com/2153359
if [ "$ARCH" = "aarch64" ]; then
if [ "$ARCH" = "aarch64" ] && ! check_clang_version_ge "16.0.0"; then
ASAN_OPTIONS="detect_leaks=0"
fi

Expand Down

0 comments on commit d19b438

Please sign in to comment.