Title: [270241] trunk/Tools
Revision
270241
Author
[email protected]
Date
2020-11-30 07:05:05 -0800 (Mon, 30 Nov 2020)

Log Message

[build.webkit.org] Update RunLLINTCLoopTests step for new buildbot
https://bugs.webkit.org/show_bug.cgi?id=219084

Reviewed by Jonathan Bedard.

* CISupport/build-webkit-org/steps.py:
(RunLLINTCLoopTests):
(RunLLINTCLoopTests.start): Initialized log observer.
(RunLLINTCLoopTests.parseOutputLine): Parse each log line as they come.
(RunLLINTCLoopTests.countFailures): Updated to return failedTestCount.

Modified Paths

Diff

Modified: trunk/Tools/CISupport/build-webkit-org/steps.py (270240 => 270241)


--- trunk/Tools/CISupport/build-webkit-org/steps.py	2020-11-30 15:00:28 UTC (rev 270240)
+++ trunk/Tools/CISupport/build-webkit-org/steps.py	2020-11-30 15:05:05 UTC (rev 270241)
@@ -868,6 +868,7 @@
     ]
     failedTestsFormatString = "%d regression%s found."
     logfiles = {"json": jsonFileName}
+    test_summary_re = re.compile(r'\s*(?P<count>\d+) regressions? found.')  # e.g.: 2 regressions found.
 
     def __init__(self, *args, **kwargs):
         kwargs['logEnviron'] = False
@@ -876,11 +877,22 @@
     def start(self):
         if USE_BUILDBOT_VERSION2:
             self.workerEnvironment[RESULTS_SERVER_API_KEY] = os.getenv(RESULTS_SERVER_API_KEY)
+            self.log_observer = ParseByLineLogObserver(self.parseOutputLine)
+            self.addLogObserver('stdio', self.log_observer)
+            self.failedTestCount = 0
         else:
             self.slaveEnvironment[RESULTS_SERVER_API_KEY] = os.getenv(RESULTS_SERVER_API_KEY)
         return shell.Test.start(self)
 
+    def parseOutputLine(self, line):
+        match = self.test_summary_re.match(line)
+        if match:
+            self.failedTestCount = int(match.group('count'))
+
     def countFailures(self, cmd):
+        if USE_BUILDBOT_VERSION2:
+            return self.failedTestCount
+
         logText = cmd.logs['stdio'].getText()
         # We're looking for the line that looks like this: 0 regressions found.
         regex = re.compile(r'\s*(?P<count>\d+) regressions? found.')

Modified: trunk/Tools/ChangeLog (270240 => 270241)


--- trunk/Tools/ChangeLog	2020-11-30 15:00:28 UTC (rev 270240)
+++ trunk/Tools/ChangeLog	2020-11-30 15:05:05 UTC (rev 270241)
@@ -1,3 +1,16 @@
+2020-11-30  Aakash Jain  <[email protected]>
+
+        [build.webkit.org] Update RunLLINTCLoopTests step for new buildbot
+        https://bugs.webkit.org/show_bug.cgi?id=219084
+
+        Reviewed by Jonathan Bedard.
+
+        * CISupport/build-webkit-org/steps.py:
+        (RunLLINTCLoopTests):
+        (RunLLINTCLoopTests.start): Initialized log observer.
+        (RunLLINTCLoopTests.parseOutputLine): Parse each log line as they come.
+        (RunLLINTCLoopTests.countFailures): Updated to return failedTestCount.
+
 2020-11-27  Wenson Hsieh  <[email protected]>
 
         Introduce new display list meta commands in preparation for webkit.org/b/219091
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to