EWS shouldn't comment on Bugzilla for failing builds or style errors
authorrniwa@webkit.org <rniwa@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 16 Jan 2014 23:15:07 +0000 (23:15 +0000)
committerrniwa@webkit.org <rniwa@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 16 Jan 2014 23:15:07 +0000 (23:15 +0000)
https://bugs.webkit.org/show_bug.cgi?id=127079

Reviewed by Anders Carlsson.

Since patch authors and reviewers could retrieve this information from bubbles,
we shouldn't spam the Bugzilla with useless comments.

We still post style errors but not python paths we used since it's a pure noise.

* Scripts/webkitpy/tool/commands/earlywarningsystem.py:
(AbstractEarlyWarningSystem._post_reject_message_on_bug):
* Scripts/webkitpy/tool/commands/queues.py:
(StyleQueue.review_patch):

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

Tools/ChangeLog
Tools/Scripts/webkitpy/tool/commands/earlywarningsystem.py
Tools/Scripts/webkitpy/tool/commands/queues.py

index ec75092e4ed98ad18e0f1eb55d3965c7df0417c5..7732bf8b0f5afbff091613d6a9ab930d66e72506 100644 (file)
@@ -1,3 +1,20 @@
+2014-01-16  Ryosuke Niwa  <rniwa@webkit.org>
+
+        EWS shouldn't comment on Bugzilla for failing builds or style errors
+        https://bugs.webkit.org/show_bug.cgi?id=127079
+
+        Reviewed by Anders Carlsson.
+
+        Since patch authors and reviewers could retrieve this information from bubbles,
+        we shouldn't spam the Bugzilla with useless comments.
+
+        We still post style errors but not python paths we used since it's a pure noise.
+
+        * Scripts/webkitpy/tool/commands/earlywarningsystem.py:
+        (AbstractEarlyWarningSystem._post_reject_message_on_bug):
+        * Scripts/webkitpy/tool/commands/queues.py:
+        (StyleQueue.review_patch):
+
 2014-01-15  Roger Fong  <roger_fong@apple.com>
 
         Add support for handling WebGL load policies.
index 709b0a3a742fc0c40e8fa3050577dc47b153cdd1..104472fa80c973a9ee9b2ad2de93b791267bf681 100644 (file)
@@ -66,6 +66,9 @@ class AbstractEarlyWarningSystem(AbstractReviewQueue, EarlyWarningSystemTaskDele
         return "New failing tests:\n%s" % "\n".join(unexpected_failures)
 
     def _post_reject_message_on_bug(self, tool, patch, status_id, extra_message_text=None):
+        if not extra_message_text:
+            return  # Don't comment on Bugzilla if we don't have failing tests.
+
         results_link = tool.status_server.results_url_for_status(status_id)
         message = "Attachment %s did not pass %s (%s):\nOutput: %s" % (patch.id(), self.name, self.port_name, results_link)
         if extra_message_text:
index 74724cffbbb4dee0f00c35e36419179c956b576e..e8c59ce6222d49f004e3662d511c74da1c38a9e0 100644 (file)
@@ -30,6 +30,7 @@
 import codecs
 import logging
 import os
+import re
 import sys
 import time
 import traceback
@@ -465,7 +466,8 @@ class StyleQueue(AbstractReviewQueue, StyleQueueTaskDelegate):
             self._did_error(patch, "%s unable to apply patch." % self.name)
             return False
         except ScriptError, e:
-            message = "Attachment %s did not pass %s:\n\n%s\n\nIf any of these errors are false positives, please file a bug against check-webkit-style." % (patch.id(), self.name, e.output)
+            output = re.sub(r'Failed to run .+ exit_code: 1', '', e.output)
+            message = "Attachment %s did not pass %s:\n\n%s\n\nIf any of these errors are false positives, please file a bug against check-webkit-style." % (patch.id(), self.name, output)
             self._tool.bugs.post_comment_to_bug(patch.bug_id(), message, cc=self.watchers)
             self._did_fail(patch)
             return False