From 8bcd2a74cac95beb0eed4d68b44e672c9149655d Mon Sep 17 00:00:00 2001 From: Josh Durgin Date: Mon, 3 Oct 2011 17:00:45 -0700 Subject: [PATCH] teuthology-results: include reasons for failure in email --- teuthology/suite.py | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/teuthology/suite.py b/teuthology/suite.py index 1ff9e33367ffd..daa8fb2afafa7 100644 --- a/teuthology/suite.py +++ b/teuthology/suite.py @@ -272,6 +272,7 @@ def results(): descriptions = [] failures = [] + num_failures = 0 unfinished = [] for j in sorted(os.listdir(args.archive_dir)): if j.startswith('.'): @@ -292,13 +293,18 @@ def results(): descriptions.append(desc) if not summary['success']: failures.append(desc) + num_failures += 1 + if 'failure_reason' in summary: + failures.append(' {reason}'.format( + reason=summary['failure_reason'], + )) if not args.email or not (failures or unfinished or args.email_on_success): return if failures or unfinished: subject = '{num_failed} failed and {num_hung} possibly hung tests in {suite}'.format( - num_failed=len(failures), + num_failed=num_failures, num_hung=len(unfinished), suite=args.name, ) -- 2.39.5