New issue
Advanced search Search tips

Issue 592673 link

Starred by 2 users

Issue metadata

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



Sign in to add a comment

Don't blame "innocent" by-standers on recipe roll CLs

Project Member Reported by tandrii@chromium.org, Mar 7 2016

Issue description

When recipe roll happens after a bunch of recipe-unrelated CLs, the reviwers are taken as a union of all authors/reviewers of CLs that contribute to the roll.  Example: https://chromereviews.googleplex.com/376607013/

IMO, only those whose CL actually lead to changes in expectations should be cc-ed.
 
Status: Available (was: Untriaged)
Makes sense, we should address this before moving recipes source side.
Labels: srcside

Comment 3 by aga...@chromium.org, Apr 26 2016

Components: Infra>Platform>Recipes
Labels: -Infra-Recipes
Labels: Pri-2
Labels: Type-Bug
Status: Fixed (was: Available)
autoroller uses `git-cl upload --r-owners` now and just cc included upstream folks.

Sign in to add a comment