Confirm
1,927
edits
Changes
BMO integration
= Phabricator at Mozilla =
In May 2017 the Engineering Operations team announced that code-review functionality in bugzilla.mozilla.org, aka BMO , and MozReview will be fully replaced by Differential,
Phabricator’s code-review tool, for Firefox and closely related products, that is, any code going into [https://hg.mozilla.org/mozilla-central mozilla-central].
= BMO Integration =
Since issue tracking and code review are tightly related, and since BMO is currently the authority for identity and authorization around both issue tracking and code review, including security and other confidential bugs and fixes, our Phabricator instance will be integrated with BMO. This integration will only be to the degree needed in order to limit customization of Phabricator, which has both maintenance and opportunity costs.
== Identity ==
The main way to log into Phabricator will be via BMO's auth delegation. A user logging into Phabricator will be taken to BMO to log in as usual and will be redirected back to Phabricator if the login succeeds. If this is the first time the user has logged into Phabricator, they will be prompted to create an account. They can choose to use their BMO email address or provide a new one, which will be separately verified. New users will also be prompted to enter a separate username, unlike BMO. This username will be used by Autoland to denote reviewers when constructing the final commit message.
== Links from Differential to BMO ==
A bug number must be entered when a patch is submitted to Phabricator. This will be stored in the revision metadata and provided in the UI as a link to the associated bug on BMO.
== Links from BMO to Differential ==
Upon the creation of a new revision in Differential, a stub attachment, containing only the URL of the revision, will be added to the associated bug. Based on the attachment type, BMO will automatically redirect to Differential if the attachment link is clicked.
== Authorization ==
If a bug has one or more security groups applied to it, that is, it has restricted visibility, any Differential revisions associated with it will similarly be restricted in visibility. This will initially only apply to Firefox security groups, that is, groups with names matching ''*core-security*''. Any revision associated with a bug restricted via other groups, e.g. infra, will be visible only to the author and admins. We can add proper support for such groups on request.
== Review status ==
For simplicity, and since Differential's review system does not map cleanly to BMO's review flags, r+ flags, and only r+ flags, will be set on the stub attachment associated with a Differential revision when a Phabricator user performs an "Accept Revision" action. The flag will be removed if the user later issues a "Request Changes" or a "Resign as Reviewer" action. Similarly, all r+ flags will be removed if the author selects any of the "Plan Changes", "Request Review", or "Abandon Revision" actions.
= FAQ =
# Deployment of a central Differential (code review) installation, supported by CloudOps, along with supporting tools like Diffusion (repository browser) and Herald (event-driven notifications). We will continue to use our Bugzilla instance, bugzilla.mozilla.org (aka BMO), for issue tracking for the foreseeable future.
# Lightweight integration of Differential with BMO. This will include
#* Authentication/identity.#* Links from Differential revisions to BMO bugs.#* Mirroring approvals (only) of Differential revisions back to BMO bugs as r+s on stub attachments, for record-keeping purposes. Aside from thisintegration, the code-review process will occur only in Differential to avoid fragmenting the discussion.#* Security-group mapping to support reviews of security and otherwise confidential patches as seamlessly as possiblediscussed above.#* Integration with and improvements to the Autoland service, which is currently used for around 50% of commits to mozilla-central and an integral part of the Quantum Stylo project.
== What will happen to Splinter and MozReview? ==
