REGRESSION: buildbot mangles error messages with timestamps run-webkit-tests outputs
authorrniwa@webkit.org <rniwa@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 11 Jan 2013 22:57:29 +0000 (22:57 +0000)
committerrniwa@webkit.org <rniwa@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 11 Jan 2013 22:57:29 +0000 (22:57 +0000)
https://bugs.webkit.org/show_bug.cgi?id=106688

Reviewed by Eric Seidel.

Update the regular expression used to strip the garbage at the beginning.

* BuildSlaveSupport/build.webkit.org-config/master.cfg:
(RunWebKitTests.nrwt_log_message_regexp):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@139505 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Tools/BuildSlaveSupport/build.webkit.org-config/master.cfg
Tools/BuildSlaveSupport/build.webkit.org-config/mastercfg_unittest.py
Tools/ChangeLog

index cbe20dd0dc063c2260bcdd6fa80f0f597a8d8230..3627af1b999f76dec484c60097cea14ebc1396dc 100644 (file)
@@ -363,7 +363,7 @@ class RunWebKitTests(shell.Test):
         self.incorrectLayoutLines = incorrectLayoutLines
 
     # FIXME: This will break if new-run-webkit-tests changes its default log formatter.
-    nrwt_log_message_regexp = re.compile(r'(?P<log_prefix>.*) (?P<log_level>DEBUG|INFO) (?P<message>.*)')
+    nrwt_log_message_regexp = re.compile(r'\d{2}:\d{2}:\d{2}(\.\d+)?\s+\d+\s+(?P<message>.*)')
 
     def _strip_python_logging_prefix(self, line):
         match_object = self.nrwt_log_message_regexp.match(line)
index 5cb6d306adf80fc48c9b1c98bcfca2ca8a2d0f40..3f86e751c91a8bc8cf03a96a991c10dec3ba53f8 100755 (executable)
@@ -53,11 +53,11 @@ class MasterCfgTest(unittest.TestCase):
     def test_nrwt_leaks_parsing(self):
         run_webkit_tests = RunWebKitTests()  # pylint is confused by the way we import the module ... pylint: disable-msg=E0602
         log_text = """
-2011-08-09 10:05:18,580 29486 mac.py:275 INFO leaks found for a total of 197,936 bytes!
-2011-08-09 10:05:18,580 29486 mac.py:276 INFO 1 unique leaks found!
+12:44:24.295 77706 13981 total leaks found for a total of 197,936 bytes!
+12:44:24.295 77706 1 unique leaks found!
 """
         expected_incorrect_lines = [
-            'leaks found for a total of 197,936 bytes!',
+            '13981 total leaks found for a total of 197,936 bytes!',
             '1 unique leaks found!',
         ]
         run_webkit_tests._parseNewRunWebKitTestsOutput(log_text)
index 54c06c6c5da66b5f9084bd7de23d68f339c46b1e..2fe4aeece8a8df2d5860c580a45ef83cd5a7244e 100644 (file)
@@ -1,3 +1,15 @@
+2013-01-11  Ryosuke Niwa  <rniwa@webkit.org>
+
+        REGRESSION: buildbot mangles error messages with timestamps run-webkit-tests outputs
+        https://bugs.webkit.org/show_bug.cgi?id=106688
+
+        Reviewed by Eric Seidel.
+
+        Update the regular expression used to strip the garbage at the beginning.
+
+        * BuildSlaveSupport/build.webkit.org-config/master.cfg:
+        (RunWebKitTests.nrwt_log_message_regexp):
+
 2013-01-11  Pratik Solanki  <psolanki@apple.com>
 
         TestRunner leaks when running tests