Wikimedia Release Engineering Team/Gerrit migration why
 | This page is obsolete. It is being retained for archival purposes. It may document extensions or features that are obsolete and/or no longer supported. Do not rely on the information here being up-to-date. Please see the RFC now in Phabricator, this document was created before that and much of it is copied into the RFC. |
Background
[edit]In early 2014, the WMF undertook an analysis of tools to use for code-review, bug management, project management, and code hosting. The impetus of this analysis was the disconnected/disjointed environment for basic development/management process across multiple platforms (at the time: Bugzilla, Gerrit, Gitblit, Trello, and Mingle). The resulting analysis can be seen in this comparison of various options, including Phabricator, Github, Scrumbugz, and the status quo (in other words: the disconnected multiple platforms).
A Request for Comments (RFC) was initiated in March of 2014 to propose the migration of all those tools to Phabricator. This RFC had a lively discussion in multiple forums (see threads on wikitech-l, teampractices list, and, of course, the RFC talk page). The RFC passed with support to begin by migrating Bugzilla to Phabricator, and if that was successful, migrating code-review (Gerrit) as well.
See, for instance, the blog post announcing Phabricator, which used this sentence: "Initially, we’ll focus on bug tracking and project management, but we’re planning to also use it for code review once the features we need have been added." In fact, the title of the blog post is also "On our way to Phabricator" implying the journey has just begun.
Where we are now
[edit]The task/project management migration happened in November 2014, with migrations from Trello and Mingle completed for all teams since then (at their own pace).
The near future
[edit]The Wikimedia Foundation's Release Engineering Team (which maintains our project management and code review platforms) is preparing to continue along the plan of migrating code-review to Phabricator during the FY1516 Q2 and Q3 quarters (iow: from October '15 to March '16).
The general outline of steps along the way is:
- Deprecate Gitblit (ie: git.wikimedia.org) and host all repositories in Phabricator ("Diffusion")
- This is in-progress already
- Migrate repositories that both 1) want to and 2) don't have CI requirements to code-review in Phabricator ("Differential")
- This is in-progress already with pywikibot, and soon some WMF RelEng owned repositories
- Doing this will give us real-world examples of code-review in Phabricator
- Setup our Continuous Integration/testing infrastructure to work with Phabricator code-review
- Share (via RFC?) our end-goal of how development and code-review will happen with Phabricator
- informed by 2.
- If no surprises, migrate code-review from Gerrit to Phabricator and make Gerrit read-only (exact details tbd)
Pros and Cons of migrating to Phabricator
[edit](partly taken from the RFC, partly new)
Pros
[edit]- Everything would be in one place
- This benefit can not be understated
- Single login for everything (it uses your Wikimedia SUL account)
- Built in integration of subsystems like bug tracker, code-review, etc
- Single interface across subsystems (reduced cognitive load/confusion)
- Integrates design and designers (via Pholio)
- Conversations allowed to stay in one place (instead of spread across multiple platforms)
- Reduce technical-debt maintained by our community by no longer maintaining a family of home-grown integration bots (see: grrrit-wm and potentially wikibugs2) and instead use/extend the Phabricator Bot.
- Reducing the technical debt maintained by the WMF by:
- Decommissioning Gerrit
- Decommissioning Gitblit
- Mobile-friendly
- Login from any mobile device and see what you can read and do. Now, open Bugzilla or Gerrit...
- Support for code auditing, with automatic notification based on rules and conditions
- Faster and much more enjoyable code review experience (especially large diffs), and improved commit/review procedures
- arc patch D123 vs git fetch https://gerrit.wikimedia.org/r/mediawiki/tools/scap refs/changes/28/239028/10 && git checkout FETCH_HEAD
- differential comments are actually readable. vs gerrit is completely illegible, and if that weren't bad enough, no amount of css can fix it because the markup is pathological. Gerrit code review: Intentionally 99% css-resistant in the 21st century.
- Multi-commit single-branch review support instead of the approach in Gerrit where each change is a single commit that is amended for each new patch set
- Better visibility of requested reviews - patches awaiting review are easier to notice thanks to dashboards, an outstanding review counter on the phabricator main menu, and phabricator notifications.
- Possibility to let users create their own repositories
Cons
[edit]- Transition learning curve (inherent to any migration)
- Can be partly addressed via trainings and documentation (which mostly already exists)
- Single Point of Failure
- Can be mitigated
- No dedicated mobile app (unlike Trello)
- It does have a mobile interface (better than Gerrit or Bugzilla)
- Repository callsigns. This is being addressed with T110607