Commit d24a5007 authored by dominicc@chromium.org's avatar dominicc@chromium.org

Pull alerts from sheriff-o-matic.appspot.com instead of auto-sheriff.

At the moment alerts are being stored both at
http://sheriff-o-matic.appspot.com/alerts and
http://auto-sheriff.appspot.com/data . auto-sheriff will be turned
down, so stop reading alerts from there.

BUG=401358
NOTRY=true

Review URL: https://codereview.chromium.org/476593002

git-svn-id: svn://svn.chromium.org/blink/trunk@180237 bbb929c8-8fbe-4397-9dbb-9b2b20218538
parent 807e2360
...@@ -57,11 +57,14 @@ While we still have tests using QUnit, you can run these at: http://localhost:80 ...@@ -57,11 +57,14 @@ While we still have tests using QUnit, you can run these at: http://localhost:80
# PUSHING # PUSHING
Fill in the revision you're pushing for XXXX. In GardeningServer:
1. appcfg.py update . --version rXXXX
2. Go to appengine.google.com and select the sheriff-o-matic app. 1. make update
3. Click on the version tab 2. appcfg.py update . --version r$(git svn find-rev HEAD) --oauth2
4. Make the rXXXX version the default 3. Go to appengine.google.com and select the sheriff-o-matic app.
4. Click on the version tab
5. Make the rXXXX version the default, where XXXX is:
git svn find-rev HEAD
The versioning is so that we can easily roll back if there's something The versioning is so that we can easily roll back if there's something
wrong with the push and so we can know what we've pushed. wrong with the push and so we can know what we've pushed.
......
...@@ -67,9 +67,7 @@ found in the LICENSE file. ...@@ -67,9 +67,7 @@ found in the LICENSE file.
update: function() { update: function() {
var annotationPromise = CTFailureGroup.fetchAnnotations(); var annotationPromise = CTFailureGroup.fetchAnnotations();
// FIXME: Change this to http://sheriff-o-matic.appspot.com/alerts net.json('http://sheriff-o-matic.appspot.com/alerts').then(function(data) {
// when Sheriff-o-matic is redeployed to serve /alerts.
net.json('http://auto-sheriff.appspot.com/data').then(function(data) {
annotationPromise.then(function(annotations) { annotationPromise.then(function(annotations) {
// FIXME: Don't special-case the blink master. // FIXME: Don't special-case the blink master.
this.builderLatestRevisions = new CTBuilderRevisions(data.latest_revisions['chromium.webkit']); this.builderLatestRevisions = new CTBuilderRevisions(data.latest_revisions['chromium.webkit']);
...@@ -109,9 +107,6 @@ found in the LICENSE file. ...@@ -109,9 +107,6 @@ found in the LICENSE file.
group.failure_keys.forEach(function(failure_key) { group.failure_keys.forEach(function(failure_key) {
var failure = failures.find(function(item) { return item.key == failure_key; }); var failure = failures.find(function(item) { return item.key == failure_key; });
if (failure.ignored_by.length)
return;
var reason, failureType; var reason, failureType;
if (failure.reason) { if (failure.reason) {
// FIXME: Store the actual failure type in a different field instead of smashing it into the reason. // FIXME: Store the actual failure type in a different field instead of smashing it into the reason.
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment