[ovs-git] [openvswitch/ovs] 574084: github: Run clang test with AddressSanitizer enabled.

Ilya Maximets noreply at github.com
Wed Feb 17 20:50:19 UTC 2021


  Branch: refs/heads/master
  Home:   https://github.com/openvswitch/ovs
  Commit: 5740843c4404caba7a38e1920fd0ed8a23a84084
      https://github.com/openvswitch/ovs/commit/5740843c4404caba7a38e1920fd0ed8a23a84084
  Author: Ilya Maximets <i.maximets at ovn.org>
  Date:   2021-02-17 (Wed, 17 Feb 2021)

  Changed paths:
    M .ci/linux-build.sh
    M .github/workflows/build-and-test.yml
    M tests/automake.mk

  Log Message:
  -----------
  github: Run clang test with AddressSanitizer enabled.

This commit is based on a similar one from OVN by Dumitru Ceara:
  a429b24f7bf5 ("ci: Enable AddressSanitizer in Linux clang CI test runs.")

It's useful to run testsuite with address sanitizer enabled to catch
memory leaks and invalid memory accesses.  Skipping re-check if
AddressSanitizer reports are present in the test run directory to
not lose them.

Right now OVS has no memory leaks detected on a testsuite run with -O1.
With -O2 there are few false-positive leak reports in test-ovsdb
application, so not using this optimization level for now.  For the
same reason not enabling leak detection by default for everyone.
Enabled only in CI.

AddressSanitizer increases execution time for this job from ~12 to ~16
minutes, but it looks like a reasonable sacrifice.

Acked-by: Dumitru Ceara <dceara at redhat.com>
Signed-off-by: Ilya Maximets <i.maximets at ovn.org>




More information about the git mailing list