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

Unified Diff: appengine/findit/crash/crash_pipeline.py

Issue 2514413003: [Predator] Monitor found_suspects, has_regression_range and found_components using ts_mon (Closed)
Patch Set: Rebase Created 4 years, 1 month 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 | appengine/findit/crash/monitoring.py » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: appengine/findit/crash/crash_pipeline.py
diff --git a/appengine/findit/crash/crash_pipeline.py b/appengine/findit/crash/crash_pipeline.py
index b0367401eb731a2e7a4c89febc8212a04c9624f2..897f0d3b4ccc7b2bef3952f565979f09240b5f8c 100644
--- a/appengine/findit/crash/crash_pipeline.py
+++ b/appengine/findit/crash/crash_pipeline.py
@@ -6,6 +6,8 @@ import copy
import json
import logging
+from crash import monitoring
+
from common import appengine_util
from common import pubsub_util
from common.http_client_appengine import HttpClientAppengine
@@ -158,6 +160,12 @@ class CrashAnalysisPipeline(CrashBasePipeline):
# TODO(http://crbug.com/659346): we misplaced the coverage test; find it!
if hasattr(analysis, tag_name): # pragma: no cover
setattr(analysis, tag_name, tag_value)
+
+ if hasattr(monitoring, tag_name):
+ metric = getattr(monitoring, tag_name)
+ metric.increment({tag_name: tag_value,
+ 'client_id': self.client_id})
+
analysis.status = analysis_status.COMPLETED
analysis.put()
« no previous file with comments | « no previous file | appengine/findit/crash/monitoring.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698