2010-12-24 Eric Seidel <eric@webkit.org>
authoreric@webkit.org <eric@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 24 Dec 2010 11:05:21 +0000 (11:05 +0000)
committereric@webkit.org <eric@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 24 Dec 2010 11:05:21 +0000 (11:05 +0000)
        Reviewed by David Levin.

        commit-queue should better explain how it will re-open bugs and follow duplicate chains
        https://bugs.webkit.org/show_bug.cgi?id=51549

        I've had multiple people ask me about the commit-queue's behavior
        regarding reporting flaky tests.  I figured it would be best to just
        have the queue explain itself in the bugs it files.

        * Scripts/webkitpy/tool/bot/flakytestreporter.py:
        * Scripts/webkitpy/tool/bot/flakytestreporter_unittest.py:

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

Tools/ChangeLog
Tools/Scripts/webkitpy/tool/bot/flakytestreporter.py
Tools/Scripts/webkitpy/tool/bot/flakytestreporter_unittest.py

index 6a2db2b..0369dd7 100644 (file)
@@ -1,3 +1,17 @@
+2010-12-24  Eric Seidel  <eric@webkit.org>
+
+        Reviewed by David Levin.
+
+        commit-queue should better explain how it will re-open bugs and follow duplicate chains
+        https://bugs.webkit.org/show_bug.cgi?id=51549
+
+        I've had multiple people ask me about the commit-queue's behavior
+        regarding reporting flaky tests.  I figured it would be best to just
+        have the queue explain itself in the bugs it files.
+
+        * Scripts/webkitpy/tool/bot/flakytestreporter.py:
+        * Scripts/webkitpy/tool/bot/flakytestreporter_unittest.py:
+
 2010-12-24  Shinichiro Hamaji  <hamaji@chromium.org>
 
         Reviewed by Eric Seidel.
index 01cbf39..56af101 100644 (file)
@@ -102,7 +102,9 @@ class FlakyTestReporter(object):
 
 The bots will update this with information from each new failure.
 
-If you would like to track this test fix with another bug, please close this bug as a duplicate.
+If you believe this bug to be fixed or invalid, feel free to close.  The bots will re-open if the flake re-occurs.
+
+If you would like to track this test fix with another bug, please close this bug as a duplicate.  The bots will follow the duplicate chain when making future comments.
 """ % format_values
 
         master_flake_bug = 50856  # MASTER: Flaky tests found by the commit-queue
index f72fb28..6685837 100644 (file)
@@ -75,7 +75,9 @@ FLAKE_MESSAGE
 
 The bots will update this with information from each new failure.
 
-If you would like to track this test fix with another bug, please close this bug as a duplicate.
+If you believe this bug to be fixed or invalid, feel free to close.  The bots will re-open if the flake re-occurs.
+
+If you would like to track this test fix with another bug, please close this bug as a duplicate.  The bots will follow the duplicate chain when making future comments.
 
 component: Tools / Tests
 cc: test@test.com
@@ -115,7 +117,9 @@ Bot: mock-bot-id  Port: MockPort  Platform: MockPlatform 1.0
 
 The bots will update this with information from each new failure.
 
-If you would like to track this test fix with another bug, please close this bug as a duplicate.
+If you believe this bug to be fixed or invalid, feel free to close.  The bots will re-open if the flake re-occurs.
+
+If you would like to track this test fix with another bug, please close this bug as a duplicate.  The bots will follow the duplicate chain when making future comments.
 
 component: Tools / Tests
 cc: abarth@webkit.org