Page MenuHomePhabricator

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

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

Details

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 requested review of this revision.Jun 29 2020, 15:50
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
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.