Chromium Code Reviews| 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, |
|
stgao
2016/11/22 00:16:30
What's the tag value? How many different values it
Sharu Jiang
2016/11/22 01:24:16
The monitored tag_value has only True and False.
|
| + 'client_id': self.client_id}) |
| + |
| analysis.status = analysis_status.COMPLETED |
| analysis.put() |