Issue 842980 link

Starred by 2 users

Issue metadata

Status:	Started
Owner:	chanli@chromium.org
Cc:	█ robert...@chromium.org chanli@chromium.org lijeffrey@chromium.org liaoyuke@chromium.org st...@chromium.org
Components:	Tools>Test>FindIt
EstimatedDays:	----
NextAction:	----
OS:	----
Pri:	3
Type:	Bug

[Findit] stop getting blame_list from build_info

Project Member Reported by chanli@chromium.org, May 15 2018

Issue description

Since now Findit is using range query to gitiles to get change logs, there's no need to keep blame_list from build_info.

Project Member

Comment 1 by bugdroid1@chromium.org, May 15 2018

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

commit 2f5237ba6888e24d7557e1a11c26f92359686785
Author: Chan <chanli@chromium.org>
Date: Tue May 15 04:56:11 2018

[Findit] fix key error for commits not in change logs.

Quick and temporary fix for https://pantheon.corp.google.com/errors/CIybwYPYmZrCbg?time=P30D&project=findit-for-me

Bug: 842980

Change-Id: Id76a2c7d600bc72504aafc0f708fb88ad6e1a24c
Reviewed-on: https://chromium-review.googlesource.com/1058726
Commit-Queue: Shuotao Gao <stgao@chromium.org>
Reviewed-by: Shuotao Gao <stgao@chromium.org>

[modify] https://crrev.com/2f5237ba6888e24d7557e1a11c26f92359686785/appengine/findit/services/compile_failure/compile_failure_analysis.py
[modify] https://crrev.com/2f5237ba6888e24d7557e1a11c26f92359686785/appengine/findit/services/ci_failure.py
[modify] https://crrev.com/2f5237ba6888e24d7557e1a11c26f92359686785/appengine/findit/services/test_failure/test_failure_analysis.py

►

Issue 842980 link

Issue metadata

[Findit] stop getting blame_list from build_info

Issue description

Comment 1 by bugdroid1@chromium.org, May 15 2018

Comment 1 Deleted

Sign in to add a comment