Page MenuHomePhabricator

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

Authored by Fabien on Mon, Jun 29, 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
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.Mon, Jun 29, 15:50
Herald added a reviewer: Restricted Project. · View Herald TranscriptMon, Jun 29, 15:50
Fabien requested review of this revision.Mon, Jun 29, 15:50
jasonbcox accepted this revision.Mon, Jun 29, 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.Mon, Jun 29, 15:59
Fabien added inline comments.Mon, Jun 29, 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.

This revision was automatically updated to reflect the committed changes.