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

Unified Diff: Tools/GardeningServer/ui/ct-failure-analyzer.html

Issue 476593002: Pull alerts from sheriff-o-matic.appspot.com instead of auto-sheriff. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 6 years, 4 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 | « Tools/GardeningServer/README ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Tools/GardeningServer/ui/ct-failure-analyzer.html
diff --git a/Tools/GardeningServer/ui/ct-failure-analyzer.html b/Tools/GardeningServer/ui/ct-failure-analyzer.html
index a1fbe2b397790802b10d4fbdd46a15d4a00b42f5..22431aabf91c4ebb8c239d4e38c3bb0c499eb2ca 100644
--- a/Tools/GardeningServer/ui/ct-failure-analyzer.html
+++ b/Tools/GardeningServer/ui/ct-failure-analyzer.html
@@ -67,9 +67,7 @@ found in the LICENSE file.
update: function() {
var annotationPromise = CTFailureGroup.fetchAnnotations();
- // FIXME: Change this to http://sheriff-o-matic.appspot.com/alerts
- // when Sheriff-o-matic is redeployed to serve /alerts.
- net.json('http://auto-sheriff.appspot.com/data').then(function(data) {
+ net.json('http://sheriff-o-matic.appspot.com/alerts').then(function(data) {
annotationPromise.then(function(annotations) {
// FIXME: Don't special-case the blink master.
this.builderLatestRevisions = new CTBuilderRevisions(data.latest_revisions['chromium.webkit']);
@@ -109,9 +107,6 @@ found in the LICENSE file.
group.failure_keys.forEach(function(failure_key) {
var failure = failures.find(function(item) { return item.key == failure_key; });
- if (failure.ignored_by.length)
- return;
-
var reason, failureType;
if (failure.reason) {
// FIXME: Store the actual failure type in a different field instead of smashing it into the reason.
« no previous file with comments | « Tools/GardeningServer/README ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698