Page MenuHomePhabricator

[CI] Run the thread sanitizer as part of the CI

Authored by Fabien on Nov 8 2019, 11:40.



ASAN and UBSAN are already part of the CI, but TSAN was missing.
There is an exception in the functional test suite for
wallet_multiwallet. This is the exact same issue than the ASAN failure
(heap used after free).
It is possible that some tests end up being flacky due to the load
overhead from TSAN (timeouts may occur), but they should be fixed as
they appear on a case to case basis.

Test Plan
ABC_BUILD_NAME=build-tsan ./contrib/teamcity/

Diff Detail

rABC Bitcoin ABC
Automatic diff as part of commit; lint not applicable.
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

Fabien created this revision.Nov 8 2019, 11:40
Herald added a reviewer: Restricted Project. · View Herald TranscriptNov 8 2019, 11:40
jasonbcox accepted this revision.Nov 8 2019, 18:21
This revision is now accepted and ready to land.Nov 8 2019, 18:21
This revision was automatically updated to reflect the committed changes.