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

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: Update names of metrics. 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') | appengine/findit/crash/monitoring.py » ('J')
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 52ef7a4d10d9942ba325519923862938ac69ac76..685e035c2d4eb2556791403e30f9c20a7ae6d547 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') | appengine/findit/crash/monitoring.py » ('J')

Powered by Google App Engine
This is Rietveld 408576698