Skip to content

Commit fa76d8d

Browse files
author
MarcoFalke
committed
test: Actually print TSan tracebacks
1 parent 7b83c7d commit fa76d8d

File tree

2 files changed

+3
-1
lines changed

2 files changed

+3
-1
lines changed

.cirrus.yml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@ env: # Global defaults
22
PACKAGE_MANAGER_INSTALL: "apt-get update && apt-get install -y"
33
MAKEJOBS: "-j10"
44
TEST_RUNNER_PORT_MIN: "14000" # Must be larger than 12321, which is used for the http cache. See https://cirrus-ci.org/guide/writing-tasks/#http-cache
5+
CI_FAILFAST_TEST_LEAVE_DANGLING: "1" # Cirrus CI does not care about dangling process and setting this variable avoids killing the CI script itself on error
56
CCACHE_SIZE: "200M"
67
CCACHE_DIR: "/tmp/ccache_dir"
78
CCACHE_NOHASHDIR: "1" # Debug info might contain a stale path if the build dir changes, but this is fine

test/functional/test_runner.py

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -589,11 +589,12 @@ def run_tests(*, test_list, src_dir, build_dir, tmpdir, jobs=1, enable_coverage=
589589
# Clean up dangling processes if any. This may only happen with --failfast option.
590590
# Killing the process group will also terminate the current process but that is
591591
# not an issue
592-
if len(job_queue.jobs):
592+
if not os.getenv("CI_FAILFAST_TEST_LEAVE_DANGLING") and len(job_queue.jobs):
593593
os.killpg(os.getpgid(0), signal.SIGKILL)
594594

595595
sys.exit(not all_passed)
596596

597+
597598
def print_results(test_results, max_len_name, runtime):
598599
results = "\n" + BOLD[1] + "%s | %s | %s\n\n" % ("TEST".ljust(max_len_name), "STATUS ", "DURATION") + BOLD[0]
599600

0 commit comments

Comments
 (0)