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

Issue 805247 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner: ----
Closed: Mar 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

[Findit] Flake Analyzer - Track actions explicitly and use them in event_reporting.

Project Member Reported by wylieb@chromium.org, Jan 24 2018

Issue description

Track actions like

BUG_CREATED
BUG_COMMENTED
CR_NOTIFICATION
etc

from findit.proto -> actions

Use them in event_reporting.py instead of implicit checking.
 
Status: Available (was: Untriaged)
Project Member

Comment 2 by bugdroid1@chromium.org, Mar 21 2018

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

commit 584da76c9322b74276ca967a6fd380b78fcfbb58
Author: Brandon Wylie <wylieb@chromium.org>
Date: Wed Mar 21 20:35:17 2018

[Findit] Flake Analyzer - Track actions explicitly

Instead of inferring information about actions from other fields,
track them explicitly in the model. Also added fields for autorevert
so they are there when the feature is added.

This feature is added in the old & new pipelines.

Bug:805247
Change-Id: I30d0bac33be9dd025d5f9d5e19d3d83f366bfc42
Reviewed-on: https://chromium-review.googlesource.com/971701
Reviewed-by: Jeffrey Li <lijeffrey@chromium.org>
Commit-Queue: Brandon Wylie <wylieb@chromium.org>

[modify] https://crrev.com/584da76c9322b74276ca967a6fd380b78fcfbb58/appengine/findit/services/test/event_reporting_test.py
[modify] https://crrev.com/584da76c9322b74276ca967a6fd380b78fcfbb58/appengine/findit/pipelines/flake_failure/test/update_monorail_bug_pipeline_test.py
[modify] https://crrev.com/584da76c9322b74276ca967a6fd380b78fcfbb58/appengine/findit/services/event_reporting.py
[modify] https://crrev.com/584da76c9322b74276ca967a6fd380b78fcfbb58/appengine/findit/pipelines/flake_failure/create_bug_for_flake_pipeline.py
[modify] https://crrev.com/584da76c9322b74276ca967a6fd380b78fcfbb58/appengine/findit/pipelines/flake_failure/test/create_bug_for_flake_pipeline_test.py
[modify] https://crrev.com/584da76c9322b74276ca967a6fd380b78fcfbb58/appengine/findit/waterfall/flake/update_flake_bug_pipeline.py
[modify] https://crrev.com/584da76c9322b74276ca967a6fd380b78fcfbb58/appengine/findit/model/flake/master_flake_analysis.py
[modify] https://crrev.com/584da76c9322b74276ca967a6fd380b78fcfbb58/appengine/findit/pipelines/flake_failure/update_monorail_bug_pipeline.py
[modify] https://crrev.com/584da76c9322b74276ca967a6fd380b78fcfbb58/appengine/findit/pipelines/flake_failure/test/notify_culprit_pipeline_test.py
[modify] https://crrev.com/584da76c9322b74276ca967a6fd380b78fcfbb58/appengine/findit/waterfall/flake/test/send_notification_for_flake_culprit_pipeline_test.py
[modify] https://crrev.com/584da76c9322b74276ca967a6fd380b78fcfbb58/appengine/findit/waterfall/flake/send_notification_for_flake_culprit_pipeline.py
[modify] https://crrev.com/584da76c9322b74276ca967a6fd380b78fcfbb58/appengine/findit/waterfall/flake/test/update_flake_bug_pipeline_test.py
[modify] https://crrev.com/584da76c9322b74276ca967a6fd380b78fcfbb58/appengine/findit/pipelines/flake_failure/notify_culprit_pipeline.py

Comment 3 by wylieb@chromium.org, Mar 22 2018

Status: Fixed (was: Available)

Sign in to add a comment