wine/gecko
0
0
Fork 0
mirror of https://gitlab.winehq.org/wine/wine-gecko.git synced 2024-09-13 09:24:08 -07:00
Code Issues Packages Projects Releases Wiki Activity

Bug 1187082 - Ensure talos always produces TALOSDATA json structure in logs so perfherder can ingest data. r=jlund

Browse Source
This commit is contained in:
Julien Pagès 2015-07-25 15:29:55 +02:00
parent 96e5beb520
commit 0fd73e1540
1 changed files with 23 additions and 14 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

37
testing/mozharness/mozharness/mozilla/testing/talos.py
View File

@ -41,9 +41,21 @@ TalosErrorList = PythonErrorList + [
class TalosOutputParser(OutputParser):
minidump_regex = re.compile(r'''talosError: "error executing: '(\S+) (\S+) (\S+)'"''')
minidump_output = None
worst_tbpl_status = TBPL_SUCCESS
def __init__(self, **kwargs):
super(TalosOutputParser, self).__init__(**kwargs)
self.minidump_output = None
self.found_talosdata = False
def update_worst_log_and_tbpl_levels(self, log_level, tbpl_level):
self.worst_log_level = self.worst_level(log_level,
self.worst_log_level)
self.worst_tbpl_status = self.worst_level(
tbpl_level, self.worst_tbpl_status,
levels=TBPL_WORST_LEVEL_TUPLE
)
def parse_single_line(self, line):
""" In Talos land, every line that starts with RETURN: needs to be
printed with a TinderboxPrint:"""
@ -53,16 +65,14 @@ class TalosOutputParser(OutputParser):
if m:
self.minidump_output = (m.group(1), m.group(2), m.group(3))
if line.startswith('INFO : TALOSDATA: '):
self.found_talosdata = True
# now let's check if buildbot should retry
harness_retry_re = TinderBoxPrintRe['harness_error']['retry_regex']
if harness_retry_re.search(line):
self.critical(' %s' % line)
self.worst_log_level = self.worst_level(CRITICAL,
self.worst_log_level)
self.worst_tbpl_status = self.worst_level(
TBPL_RETRY, self.worst_tbpl_status,
levels=TBPL_WORST_LEVEL_TUPLE
)
self.update_worst_log_and_tbpl_levels(CRITICAL, TBPL_RETRY)
return # skip base parse_single_line
super(TalosOutputParser, self).parse_single_line(line)
@ -594,6 +604,10 @@ class Talos(TestingMixin, MercurialScript, BlobUploadMixin):
self.info("Looking at the minidump files for debugging purposes...")
for item in parser.minidump_output:
self.run_command(["ls", "-l", item])
if not parser.found_talosdata:
self.critical("No talos data in output!")
parser.update_worst_log_and_tbpl_levels(WARNING, TBPL_WARNING)
if self.return_code not in [0]:
# update the worst log level and tbpl status
log_level = ERROR
@ -605,12 +619,7 @@ class Talos(TestingMixin, MercurialScript, BlobUploadMixin):
log_level = WARNING
tbpl_level = TBPL_RETRY
parser.worst_log_level = parser.worst_level(
log_level, parser.worst_log_level
)
parser.worst_tbpl_status = parser.worst_level(
tbpl_level, parser.worst_tbpl_status,
levels=TBPL_WORST_LEVEL_TUPLE
)
parser.update_worst_log_and_tbpl_levels(log_level, tbpl_level)
self.buildbot_status(parser.worst_tbpl_status,
level=parser.worst_log_level)