On 2017-03-09 5:19 PM, Lawrence Mandel wrote:
> On Thu, Mar 9, 2017 at 5:14 PM, L. David Baron wrote:
>> On Thursday 2017-03-09 16:53 -0500, Mike Connor wrote:
>>> I've identified the following goals as critical for a responsible commit
>>> access policy:
>>> - The change that gets pushed must be the same change that was
>> I'm curious what this goal means. In particular, does it mean that
>> you're trying to end "review+ if you make the following changes",
>> and require that reviewers re-review the revisions no matter what?
>> (If it does mean that, then that's a substantial increase on
>> reviewer load; if it doesn't, then I'm curious what definition of
>> "the same" you're using.)
>>> In order to achieve these goals, I propose that we commit to making the
>>> following changes to all Firefox product repositories:
>>> - Direct commit access to repositories will be strictly limited to
>>> sheriffs and a subset of release engineering.
>>> - Any direct commits by these individuals will be limited to fixing
>>> bustage that automation misses and handling branch merges.
>>> - All other changes will go through an autoland-based workflow.
>>> - Developers commit to a staging repository, with scripting that
>>> connects the changeset to a Bugzilla attachment, and integrates
>>> with review
>>> - Reviewers and any other approvers interact with the changeset as
>>> today (including ReviewBoard if preferred), with Bugzilla flags as
>>> canonical source of truth.
>>> - Upon approval, the changeset will be pushed into autoland.
>>> - If the push is successful, the change is merged to
>>> and the bug updated.
>> I'm curious if this will mean that ReviewBoard will be required, or
>> if it will still be a way to use attachment-based workflows. (I ask
>> this because I still consider the ReviewBoard UI unacceptable for
>> changes that are likely to require re-review. See
>> https://bugzilla.mozilla.org/show_bug.cgi?id=1285874#c20 for my form
>> response on the topic, although I've been a little less picky about
>> requiring attachments in all cases lately, when I think things
>> aren't likely to require multiple rounds of review.)
> Mark Cote's team is currently working on enabling autoland from Bugzilla so
> MozReview will not be required.
Also fwiw we have (finally) made some progress on the bugs you mentioned