New issue
Advanced search Search tips

Issue 691474 link

Starred by 0 users

Issue metadata

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



Sign in to add a comment

Sheriff-Chromium label or Hotlist-Sheriff-Chromium label

Project Member Reported by tyoshino@chromium.org, Feb 13 2017

Issue description

Sheriff-o-Matic's "File bug" link on the "Enter bug number" dialog which opens when we click "LINK/FILE BUG" button adds "sheriff-chromium" label.

But crbug's standard label list doesn't contain it. It has "Hotlist-Sheriff-Chromium" instead.

http://www.chromium.org/developers/tree-sheriffs/sheriffing-bug-queues is also using "Sheriff-Chromium".

We should converge them, I think.
 
Cc: aga...@chromium.org
Aaron, where do I find out who manages the list of labels on Chromium project in Monorail? I can see the list of labels, but no obvious way to edit them. Would be nice to have an email address listed somewhere, so that we don't have to bother Monorail team for this...

Comment 3 by aga...@chromium.org, Feb 13 2017

Cc: -aga...@chromium.org
Labels: Pri-2 Type-Bug
Components: Admin>LabelRequest
Please rename Hotlist-Sheriff-Chromium label into Sheriff-Chromium.
Status: Untriagged (was: Available)
Labels: -Restrict-View-Google
Removing Restrict-View-Google as nothing confidential is discussed here.

Comment 8 by mgiuca@chromium.org, Feb 16 2017

Labels: -Hotlist-Sheriff-Chromium
Removing Hotlist-Sheriff-Chromium. This is not an issue for sheriffs to deal with.

Comment 9 by laforge@google.com, Feb 28 2017

Components: -Admin>LabelRequest
Added Sheriff-Chromium and removed Hotlist-Sheriff-Chromium from auto-complete.  Please update/ bulk move the issues, this should also unblock a tooling change. 

As a general comment, creating new label namespaces should be a last resort, especially when they are only populated by a single leaf.  The reason that I added it, in this case, is that having a spot for sheriff teams/ labels would be good general practice and might encourage additional usage.

Labels: Sheriff-Chromium
Please use Sheriff-Chromium label instead of Hotlist-Sheriff-Chromium. Issues with Sheriff-Chromium label are shown in Sheriff-o-Matic and are tried by sheriffs directly.
Labels: -Sheriff-Chromium
Turns out search on Monorail is eventually consistent. Removing Sheriff-Chromium from this issue since Hotlist-Sheriff-Chromium was removed earlier.
Components: -Infra>Sheriffing>SheriffOMatic
Sheriff-o-Matic is already using the correct component in this case.
Cc: lafo...@chromium.org
Labels: Infra-Troopers
Owner: serg...@chromium.org
Status: Fixed (was: Untriagged)
Re general comment from #9: we are now planning to add Sheriff-WebRTC and in future other labels like Sheriff-V8 will follow. The only label which falls short of this pattern is Infra-Troopers, which is used as a queue for troopers. IMHO, it would be nice to rename this to Sheriff-Infra, but this is a non-trivial change as it requires inspecting systems that depend on label to Infra-Troopres and updating those systems.

Sign in to add a comment