Title: [91701] trunk/Tools
Revision
91701
Author
o...@chromium.org
Date
2011-07-25 13:08:33 -0700 (Mon, 25 Jul 2011)

Log Message

don't show builders in the flakiness dashboard that have stopped running a given test suite
https://bugs.webkit.org/show_bug.cgi?id=65126

Reviewed by Mihai Parparita.

* TestResultServer/static-dashboards/dashboard_base.js:

Modified Paths

Diff

Modified: trunk/Tools/ChangeLog (91700 => 91701)


--- trunk/Tools/ChangeLog	2011-07-25 20:05:28 UTC (rev 91700)
+++ trunk/Tools/ChangeLog	2011-07-25 20:08:33 UTC (rev 91701)
@@ -1,3 +1,12 @@
+2011-07-25  Ojan Vafai  <o...@chromium.org>
+
+        don't show builders in the flakiness dashboard that have stopped running a given test suite
+        https://bugs.webkit.org/show_bug.cgi?id=65126
+
+        Reviewed by Mihai Parparita.
+
+        * TestResultServer/static-dashboards/dashboard_base.js:
+
 2011-07-25  Adam Barth  <aba...@webkit.org>
 
         Overhaul garden-o-matic frontend to integrate revision history and failures

Modified: trunk/Tools/TestResultServer/static-dashboards/dashboard_base.js (91700 => 91701)


--- trunk/Tools/TestResultServer/static-dashboards/dashboard_base.js	2011-07-25 20:05:28 UTC (rev 91700)
+++ trunk/Tools/TestResultServer/static-dashboards/dashboard_base.js	2011-07-25 20:08:33 UTC (rev 91701)
@@ -108,6 +108,7 @@
 var TIMESTAMPS_KEY = 'secondsSinceEpoch';
 var BUILD_NUMBERS_KEY = 'buildNumbers';
 var TESTS_KEY = 'tests';
+var TWO_WEEKS_SECONDS = 60 * 60 * 24 * 14;
 
 // These should match the testtype uploaded to test-results.appspot.com.
 // See http://test-results.appspot.com/testfile.
@@ -470,8 +471,18 @@
 function ADD_RESULTS(builds)
 {
     for (var builderName in builds) {
-        if (builderName != 'version')
-            g_resultsByBuilder[builderName] = builds[builderName];
+        if (builderName == 'version')
+            continue;
+
+        // If a test suite stops being run on a given builder, we don't want to show it.
+        // Assume any builder without a run in two weeks for a given test suite isn't
+        // running that suite anymore.
+        // FIXME: Grab which bots run which tests directly from the buildbot JSON instead.
+        var lastRunSeconds = builds[builderName].secondsSinceEpoch[0];
+        if ((Date.now() / 1000) - lastRunSeconds > TWO_WEEKS_SECONDS)
+            continue;
+
+        g_resultsByBuilder[builderName] = builds[builderName];
     }
 
     handleResourceLoad();
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to