Differential D6767

[CI] Log sanitizers output to stdout instead of log files
ClosedPublic
Actions

Authored by Fabien on Jun 29 2020, 15:50.

Details

Reviewers

jasonbcox

Group Reviewers

Restricted Project

Commits

rABCdbf3ab2a2f44: [CI] Log sanitizers output to stdout instead of log files

Summary

This will allow to correlate the output with the running test timewise,
which can be helpful during debugging.

Test Plan

./contrib/teamcity/build-configurations.py build-asan
./contrib/teamcity/build-configurations.py build-ubsan
./contrib/teamcity/build-configurations.py build-tsan

Diff Detail

Repository

rABC Bitcoin ABC

Branch

ci_log_stdout

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 11766
Build 23122: Default Diff Build & Tests	Link to the build
Build 23121: Build without Wallet	Link to the build
Build 23120: Build with clang-10	Link to the build
Build 23119: arc lint + arc unit

Event Timeline

Fabien created this revision.Jun 29 2020, 15:50

Herald added a reviewer: Restricted Project. · View Herald TranscriptJun 29 2020, 15:50

Fabien requested review of this revision.Jun 29 2020, 15:50

jasonbcox accepted this revision.Jun 29 2020, 15:59

jasonbcox added a subscriber: jasonbcox.

jasonbcox added inline comments.

contrib/teamcity/ci-fixture.sh
31	Not really related to this diff, but should suppressions also be moved into the config?

This revision is now accepted and ready to land.Jun 29 2020, 15:59

Fabien added inline comments.Jun 29 2020, 16:04

contrib/teamcity/ci-fixture.sh
31	Suppression files are already set by cmake. The only reason for having it here is that we are not calling a ninja target. If it is moved to the configuration it would end up being duplicated for all the ninja calls.