From 5b4c6aa97f3ce87585ba940642dc3eaa2fef7a12 Mon Sep 17 00:00:00 2001 From: Adam Williamson Date: Mar 31 2016 03:56:24 +0000 Subject: check-compose: count passes and fails by arch in text report this just makes it easier to see at a glance when i386 is completely titsup, really. --- diff --git a/check-compose b/check-compose index 5cc02f5..c33ddc1 100755 --- a/check-compose +++ b/check-compose @@ -26,6 +26,7 @@ import os import smtplib import sys import time +from collections import Counter from operator import attrgetter import fedfind.release @@ -341,9 +342,15 @@ def generate_reports(rel, prevrel=None, wait=0, server='', url=''): fails = [job for job in jobs if job['result'] == 'failed'] passes = [job for job in jobs if job['result'] == 'passed'] skips = [job for job in jobs if job['result'] == 'skipped'] + jcnt = Counter([job['settings']['ARCH'] for job in jobs]) if fails: gotfail = True - text += "\n\nFailed openQA tests: {0} of {1}\n".format(str(len(fails)), str(len(jobs))) + text += "\n\nFailed openQA tests: " + fcnt = Counter([fail['settings']['ARCH'] for fail in fails]) + farchs = [] + for arch in fcnt: + farchs.append("{0}/{1} ({2})".format(fcnt[arch], jcnt[arch], arch)) + text += ', '.join(farchs) + "\n" for fail in fails: settings = fail['settings'] testname = ' '.join( @@ -354,7 +361,12 @@ def generate_reports(rel, prevrel=None, wait=0, server='', url=''): text += "\nID: {0}\tTest: {1}".format(fail['id'], testname) text += "\nURL: {0}{1}{2}".format(url, "/tests/", fail['id']) if passes: - text += "\n\nPassed openQA tests: {0} of {1}".format(str(len(passes)), str(len(jobs))) + text += "\n\nPassed openQA tests: " + pcnt = Counter([pas['settings']['ARCH'] for pas in passes]) + parchs = [] + for arch in pcnt: + parchs.append("{0}/{1} ({2})".format(pcnt[arch], jcnt[arch], arch)) + text += ', '.join(parchs) + "\n" if skips: text += "\nSkipped openQA tests: {0} of {1}".format(str(len(skips)), str(len(jobs))) others = len(jobs) - (len(passes) + len(fails) + len(skips))