New issue
Advanced search Search tips

Issue 713873 link

Starred by 1 user

Issue metadata

Status: Untriaged
Owner: ----
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: ----
Type: ----


Previous locations:
gerrit:6038


Sign in to add a comment

Adding TBR in gerrit didn't work

Project Member Reported by thakis@chromium.org, Apr 20 2017

Issue description

Affected Version:

What steps will reproduce the problem?
1. Write urgent build fix CL, e.g. https://chromium-review.googlesource.com/c/483683/
2. Hit CQ (after +1'ing the patch yourself)
3. Get "presubmit failed, needs lgtm"
4. hit "Edit" next to CL description, add "TBR=dpranke"
5. Hit CQ again.

What is the expected output?

Change lands.


What do you see instead?

Presubmit fails again with "missing lgtm from owner".

(see red run 1 and 2 on the above-linked change)


Please provide any additional information below.

 

Comment 1 by logan@google.com, Apr 20 2017

Components: -PolyGerrit
Labels: Proj-Gerrit-Migration
This seems like a policy/configuration issue, so passing this to chrome-infra. Feel free to move back if gerrit help is needed to resolve this.

Comment 2 by logan@google.com, Apr 20 2017

Project: chromium
Moved issue gerrit:6038 to now be issue chromium:713873.
Status: Untriaged (was: New)
Issue moved from a project with a different set of status labels. "New" is not a supported status in /p/chromium, so these ended up in a black-hole.

Sign in to add a comment