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

Issue 719696 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: May 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug

Blocking:
issue 719618



Sign in to add a comment

[Findit] use named-tasks to create reverts

Project Member Reported by chanli@chromium.org, May 8 2017

Issue description

To avoid multiple findit analyses trying to revert the same culprit at the same time, use named-tasks for each revert (culprit revision as name).
 
Blocking: 719618
Project Member

Comment 2 by bugdroid1@chromium.org, May 16 2017

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

commit b1dcd6b1a9e0389ef7efe8fc34a5316719edf76a
Author: Chan <chanli@chromium.org>
Date: Tue May 16 22:42:28 2017

Record pipeline id when creating revert for culprit.

Record pipeline id when creating revert of a culprit to avoid Findit analyses competing with each other.

Bug:719696
Change-Id: If51e2ecc840d8d368190842017bbbf363cbefad1
Reviewed-on: https://chromium-review.googlesource.com/505395
Commit-Queue: Chan Li <chanli@chromium.org>
Reviewed-by: Shuotao Gao <stgao@chromium.org>
Reviewed-by: Jeffrey Li <lijeffrey@chromium.org>

[modify] https://crrev.com/b1dcd6b1a9e0389ef7efe8fc34a5316719edf76a/appengine/findit/waterfall/create_revert_cl_pipeline.py
[modify] https://crrev.com/b1dcd6b1a9e0389ef7efe8fc34a5316719edf76a/appengine/findit/model/base_suspected_cl.py
[modify] https://crrev.com/b1dcd6b1a9e0389ef7efe8fc34a5316719edf76a/appengine/findit/waterfall/test/create_revert_cl_pipeline_test.py

Comment 3 by chanli@chromium.org, May 16 2017

Status: Fixed (was: Assigned)

Sign in to add a comment