Page MenuHomePhabricator

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

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

Details

Summary

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/build-configurations.sh

Diff Detail

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

Event Timeline

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