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

Lint

Lint Not Applicable

Unit

Tests Not Applicable

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 ↗	(On Diff #21852)	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 ↗	(On Diff #21852)	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.