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