New issue
Advanced search Search tips

Issue 705074 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Closed: Apr 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: ----



Sign in to add a comment

Whitelist /api/v1/alerts/chromium to use cron-generated alerts by default

Project Member Reported by seanmccullough@chromium.org, Mar 24 2017

Issue description

https://sheriff-o-matic.appspot.com/chromium has been looking pretty close to https://sheriff-o-matic.appspot.com/chromium?useMilo 

android, ios still pretty far off though. We should have /chromium start using these GAE cron alerts instead of the alerts-dispatcher generated ones.


 
Project Member

Comment 1 by bugdroid1@chromium.org, Mar 24 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/infra/infra/+/5e5521de3cb9c1b2b917c7d4d1148cc1d7b675ec

commit 5e5521de3cb9c1b2b917c7d4d1148cc1d7b675ec
Author: Sean McCullough <seanmccullough@chromium.org>
Date: Fri Mar 24 20:55:57 2017

[som] Default /chromium to use the new GAE cron alerts instead of A-D

Bug:  705074 
Change-Id: I5f9f77c7311e6ff56bcde598b2c998053c826d54
Reviewed-on: https://chromium-review.googlesource.com/459081
Reviewed-by: Tiffany Zhang <zhangtiff@chromium.org>
Commit-Queue: Sean McCullough <seanmccullough@chromium.org>

[modify] https://crrev.com/5e5521de3cb9c1b2b917c7d4d1148cc1d7b675ec/go/src/infra/appengine/sheriff-o-matic/som/alerts.go

Project Member

Comment 2 by bugdroid1@chromium.org, Mar 24 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/infra/infra/+/5e5521de3cb9c1b2b917c7d4d1148cc1d7b675ec

commit 5e5521de3cb9c1b2b917c7d4d1148cc1d7b675ec
Author: Sean McCullough <seanmccullough@chromium.org>
Date: Fri Mar 24 20:55:57 2017

[som] Default /chromium to use the new GAE cron alerts instead of A-D

Bug:  705074 
Change-Id: I5f9f77c7311e6ff56bcde598b2c998053c826d54
Reviewed-on: https://chromium-review.googlesource.com/459081
Reviewed-by: Tiffany Zhang <zhangtiff@chromium.org>
Commit-Queue: Sean McCullough <seanmccullough@chromium.org>

[modify] https://crrev.com/5e5521de3cb9c1b2b917c7d4d1148cc1d7b675ec/go/src/infra/appengine/sheriff-o-matic/som/alerts.go

Status: Fixed (was: Started)

Sign in to add a comment