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

Issue 667381 link

Starred by 1 user

Issue metadata

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

Blocked on:
issue 617952



Sign in to add a comment

Can't provide a Code-Review score for CLs in tools/build

Project Member Reported by eakuefner@chromium.org, Nov 21 2016

Issue description

I used to be able to stamp things in tools/build (and indeed am an OWNER for master.chromium.perf and master.chromium.perf.fyi) but now it seems like I can't vote on a Code-Review score, e.g. in https://chromium-review.googlesource.com/c/412502/

If you use the legacy UI, the tooltips when you hover over reviewers' names show you the voting privileges people have; Emily and I seem to only have Commit-Queue but stip has both Code-Review and Commit-Queue. When I click the reply button, I see no radio buttons for Code-Review, only Commit-Queue.
 
Cc: aga...@chromium.org
Aaron, I feel like this is a known issue?
Components: Infra>Git>Admin
Owner: aga...@chromium.org
Status: Assigned (was: Untriaged)

Comment 4 by aga...@chromium.org, Nov 21 2016

Blockedon: 617952
Yes, this is due to the lack of sync between chrome-infra-auth (which provided the groups backend for CQ, which was the only thing checking permissions on Rietveld) and Gerrit, which proactively checks permissions before CQ even has to.

You are a member of build-master-owners, which gets inherited into project-infra-committers, so that your LGTMs are counted by the CQ. Unfortunately, only the latter group (and not the inheritance of build-master-owners) is currently sync'd.

We're working on fixing this. Blocking on tracking issue.

Comment 5 by aga...@chromium.org, Nov 21 2016

Components: -Infra>Git>Admin
Ping - please provide an update to your high priority bug. This bug is stale. Is it really P-1?
Status: Fixed (was: Assigned)
The blocking issue has been fixed, and this should be fixed as well. We've also improved the UI messaging when you don't have sufficient privs to set a label.

Sign in to add a comment