Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1697)

Unified Diff: Tools/GardeningServer/model/ct-failures.html

Issue 661463004: Include all hung bots alerts in Trooper tree. (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Remove comment and fix var names. Created 6 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | Tools/GardeningServer/model/test/ct-failures-tests.html » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Tools/GardeningServer/model/ct-failures.html
diff --git a/Tools/GardeningServer/model/ct-failures.html b/Tools/GardeningServer/model/ct-failures.html
index 9ddbcbdee1f8df7c37e18e1325900b80d731178b..a8efecdc662ec8cad18c18b836aed4a9ac4ad86a 100644
--- a/Tools/GardeningServer/model/ct-failures.html
+++ b/Tools/GardeningServer/model/ct-failures.html
@@ -134,7 +134,7 @@ CTFailures.prototype.update = function() {
}.bind(this));
this.failures = updateUtil.updateLeft(this.failures, newFailures);
- this._processTrooperFailures(trooper_data);
+ this._processTrooperFailures(newFailures, trooper_data);
}.bind(this));
};
@@ -150,13 +150,23 @@ CTFailures.prototype._failureComparator = function(a, b) {
return 0;
};
-CTFailures.prototype._processTrooperFailures = function(data) {
+CTFailures.prototype._processTrooperFailures = function(all_failures,
+ trooper_data) {
var trooper_failures = [];
- Object.keys(data, function(failureType, failuresByTree) {
+ Object.keys(trooper_data, function(failureType, failuresByTree) {
Object.keys(failuresByTree, function(tree, failure) {
if (failure.should_alert) {
trooper_failures.push(new CTFailureGroup('trooper',
- new CTTrooperFailureGroupData(failure.details, failure.url, failure, failureType, tree)));
+ new CTTrooperFailureGroupData(failure.details, failure.url, failure,
+ failureType, tree)));
+ }
+ });
+ });
+
+ Object.keys(all_failures, function(tree, failuresByTree) {
+ failuresByTree.forEach(function(failure) {
+ if (failure.category == 'builder' || failure.category == 'master') {
+ trooper_failures.push(failure);
}
});
});
« no previous file with comments | « no previous file | Tools/GardeningServer/model/test/ct-failures-tests.html » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698