New issue
Advanced search Search tips

Issue 831427 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Apr 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 1
Type: Bug-Regression



Sign in to add a comment

Gerrit CL doesn't show in trybots migrated to LUCI

Project Member Reported by ynovikov@chromium.org, Apr 11 2018

Issue description

Labels: -Pri-2 Pri-1
Summary: Gerrit CL doesn't show in trybots migrated to LUCI (was: Gerrit CL doesn't show in ANGLE trybots migrated to LUCI)
Actually, this is broken for Chrome trybots as well, for example
https://ci.chromium.org/p/chromium/builders/luci.chromium.try/android_optional_gpu_tests_rel/1667
Project Member

Comment 2 by bugdroid1@chromium.org, Apr 20 2018

The following revision refers to this bug:
  https://chromium.googlesource.com/infra/luci/luci-go.git/+/21e8e5ab3375f2153579cd31d92f41b53273801a

commit 21e8e5ab3375f2153579cd31d92f41b53273801a
Author: Ryan Tseng <hinoka@google.com>
Date: Fri Apr 20 22:24:20 2018

[milo] Fix blamelist issues with buildbucket builds

This fixes a couple issues with commits and blamelists:
* Don't display the "capped at 50 commits" when there are 0 matching commits.
* Display patch information on tryjobs

Bug:  831427 
Change-Id: Iaad7464e5b4b5e7a56b60c2e1b15889ca701b53d
Reviewed-on: https://chromium-review.googlesource.com/1015233
Reviewed-by: Andrii Shyshkalov <tandrii@chromium.org>
Commit-Queue: Ryan Tseng <hinoka@chromium.org>

[modify] https://crrev.com/21e8e5ab3375f2153579cd31d92f41b53273801a/milo/buildsource/buildbucket/build.go
[modify] https://crrev.com/21e8e5ab3375f2153579cd31d92f41b53273801a/milo/buildsource/buildbucket/expectations/master.tryserver.infra/InfraPresubmit.Swarming.json

Owner: hinoka@chromium.org
Status: Fixed (was: Untriaged)

Sign in to add a comment