Page MenuHomePhabricator
Differential D4412

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

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

Details

Reviewers
deadalnix
jasonbcox
Group Reviewers
Restricted Project
Commits
rSTAGING5e0d8de68df6: [CI] Run the thread sanitizer as part of the CI
rABC5e0d8de68df6: [CI] Run the thread sanitizer as part of the CI
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
Branch
ci_tsan
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 8032
Build 14060: Bitcoin ABC Buildbot (legacy)
Build 14059: arc lint + arc unit

Event Timeline

Fabien created this revision.Nov 8 2019, 11:40
Herald added a reviewer: Restricted Project. · View Herald TranscriptNov 8 2019, 11:40
Herald added a reviewer: deadalnix. · View Herald Transcript
teamcity added a comment.Nov 8 2019, 11:40

Build Bitcoin-ABC / Diffs / Diff Testing started.

teamcity added a comment.Nov 8 2019, 11:50

Build Bitcoin-ABC / Diffs / Diff Testing passed.

Harbormaster completed remote builds in B8032: Diff 13985.Nov 8 2019, 11:50
teamcity added a comment.Nov 8 2019, 11:57

Build Bitcoin-ABC / Diffs / Diff Testing started.

teamcity added a comment.Nov 8 2019, 12:05

Build Bitcoin-ABC / Diffs / Diff Testing failed.

teamcity added a comment.Nov 8 2019, 12:17

Build Bitcoin-ABC / Diffs / Diff Testing started.

teamcity added a comment.Nov 8 2019, 12:24

Build Bitcoin-ABC / Diffs / Diff Testing failed.

teamcity added a comment.Nov 8 2019, 12:26

Build Bitcoin-ABC / Diffs / Diff Testing started.

teamcity added a comment.Nov 8 2019, 12:33

Build Bitcoin-ABC / Diffs / Diff Testing failed.

teamcity added a comment.Nov 8 2019, 17:02

Build Bitcoin-ABC / Diffs / Diff Testing started.

teamcity added a comment.Nov 8 2019, 17:13

Build Bitcoin-ABC / Diffs / Diff Testing failed.

teamcity added a comment.Nov 8 2019, 17:20

Build Bitcoin-ABC / Diffs / Diff Testing started.

teamcity added a comment.Nov 8 2019, 17:34

Build Bitcoin-ABC / Diffs / Diff Testing failed.

teamcity added a comment.Nov 8 2019, 17:45

Build Bitcoin-ABC / Diffs / Diff Testing started.

teamcity added a comment.Nov 8 2019, 18:14

Build Bitcoin-ABC / Diffs / Diff Testing passed.

jasonbcox accepted this revision.Nov 8 2019, 18:21
This revision is now accepted and ready to land.Nov 8 2019, 18:21
Closed by commit rABC5e0d8de68df6: [CI] Run the thread sanitizer as part of the CI (authored by Fabien). · Explain WhyNov 8 2019, 18:28
This revision was automatically updated to reflect the committed changes.
Fabien added a commit: rABC5e0d8de68df6: [CI] Run the thread sanitizer as part of the CI.
Fabien added a commit: rSTAGING5e0d8de68df6: [CI] Run the thread sanitizer as part of the CI.Apr 7 2020, 21:39

Revision Contents
Changeset List

PathSize
contrib/
teamcity/
9 lines

Diff 13985

View Options

contrib/teamcity/build-configurations.sh

Loading...