New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 789628 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Last visit > 30 days ago
Closed: Nov 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 1
Type: Bug

Blocking:
issue 774270


Show other hotlists

Hotlists containing this issue:
XXX


Sign in to add a comment

Move http_alerts to chrome-ops-monitoring

Project Member Reported by katthomas@chromium.org, Nov 29 2017

Issue description

Some files (ie: appengine_alerts.py) include shared alert code in the same files where alerts are defined rather than separating out common code into a separate library. 

This creates a web of dependencies between different alert files across different owners/services, which is a maintenance burden. Developers may edit shared alert code without realizing that the code is being used elsewhere. 

This move is a prerequisite for the trooper rotation split
 
Status: Started (was: Assigned)
Status: Fixed (was: Started)

Sign in to add a comment