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

Issue 729149 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Last visit > 30 days ago
Closed: Jan 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

Fix Predator metric

Project Member Reported by st...@chromium.org, Jun 2 2017

Issue description

As Aaron suggested (many thanks!), the metric should be as below instead of those in predator/common/monitoring.py

reports_processed = gae_ts_mon.CounterMetric(
    'predator/reports_count',
    'Metric counting the number of crash reports that Predator has processed. '
    'Contains fields describing whether Predator was successful at finding a '
    'regression range, a components, or suspect changes for each report.',
    [gae_ts_mon.BooleanField('found_suspects'),
     gae_ts_mon.BooleanField('found_components'),
     gae_ts_mon.BooleanField('has_regression_range'),
     gae_ts_mon.StringField('client_id')])
 
Labels: -Pri-1 Pri-3
De-prioritize the bug since it's not urgent.
Labels: -Pri-3 Pri-2
Labels: -Pri-2 Pri-3
Project Member

Comment 4 by bugdroid1@chromium.org, Jan 3 2018

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

commit 45c1a38001a3e900766f4eb743749be60ddb1c1f
Author: Sharu Jiang <katesonia@google.com>
Date: Wed Jan 03 19:17:07 2018

[Predator] Modify monitoring metrics.

Bug:  729149 
Change-Id: Ib86d89d01455435a85ed0d167122ac5a5a5dc867
Reviewed-on: https://chromium-review.googlesource.com/848304
Commit-Queue: Sharu Jiang <katesonia@chromium.org>
Reviewed-by: Shuotao Gao <stgao@chromium.org>

[modify] https://crrev.com/45c1a38001a3e900766f4eb743749be60ddb1c1f/appengine/predator/app/common/monitoring.py
[modify] https://crrev.com/45c1a38001a3e900766f4eb743749be60ddb1c1f/appengine/predator/app/common/crash_pipeline.py

Status: Fixed (was: Assigned)

Sign in to add a comment