Page MenuHomePhabricator

Migrate Pywikibot to Differential code review
Closed, DeclinedPublic


WMF projects are moving from Gerrit to Differential. Pywikibot team should plan our migration, trialling the Differential code review, comparing it to other options, and feeding the trial results into the public RFC to help inform the broader Pywikibot community decision making.

Related Objects

Resolved Cmjohnson
Resolved demon
Resolved demon
Resolved demon
Resolved mmodell
Resolved demon

Event Timeline

jayvdb raised the priority of this task from to Needs Triage.
jayvdb updated the task description. (Show Details)
jayvdb added projects: Phabricator, Pywikibot.
jayvdb added subscribers: jayvdb, Qgil, mmodell.
Restricted Application added subscribers: Aklapper, Unknown Object (MLST). · View Herald TranscriptApr 9 2015, 6:52 AM

I guess we can take it for granted that Differential will be easier to use than Gerrit, at least for casual browsing/commenting/etc. (Pushing code up for review may be less easy, for various code cutting workflows/host platforms/etc)

One of the other main benefits I foresee is being able to use phab search to find design decisions made in review comments.

github search can do this; e.g. might do this, but I am guessing that it will be hard to use.

As a result, it would be a killer feature if we could import our old gerrit discussions into Differential. pywikibot has moved around quite a lot in recent years, making it quite hard to find relevant discussions about complex issues.
Extracting the old gerrit discussions isnt too difficult using the Gerrit API; pushing them into Differential might be more difficult.

Importing gerrit comments into differential will be a huge ordeal. I'm not sure a final decision has been made on this. IMO we should not migrate gerrit review history in order to avoid a whole lot of migration hell.

It's not at all easy to migrate reviews and their comments into differential. It will take a dedicated team many many hours to pull it off, it's similar amount of work to the Bugzilla migration and very little of the work that went into that migration can be reused for differential.

Has importing old reviews been attempted at all? I'd be happy to do all the work involved for pywikibot-core if given access or someone with access will review the import script and run it in if there are no issues with it.

@Ricordisamoa , I saw you did a review ; could you add a link to it here?

Guess in this case we will host the repo at phab instead of gerrit primary?

Aklapper triaged this task as Medium priority.Feb 9 2016, 10:26 PM
greg subscribed.

Removing a couple projects to clean this up. Let us (Release-Engineering-Team) know if you need any help with this migration (iow: when you want us to help you do it). To watch the progress on the overall project of migrating code-review to Gerrit, see Gerrit-Migration. (NB: I removed that project from here because except for odd circumstances we won't have a task per repository that will be migrated, in that project, as most will be en masse.)

(NB: I removed that project from here because except for odd circumstances we won't have a task per repository that will be migrated, in that project, as most will be en masse.)

I undid that part and made a "repository migrations" column on the Gerrit-Migration board. Sorry for the noise.

The statement on the description is no longer true. MediaWiki development will continue on Gerrit after the Gerrit-Migration project has been abandoned and the related RfC on was closed as declined as well. Differential is currently not recommended for code review, and I personally find it harder to work with than Gerrit. In the meanwhile Gerrit has evolved into a much usable platform with gitiles for code view and the posibility of using the web UI to create and edit patches as well. I do not see pressing reasons to force a migration here right now, moreover when also our RfC on pywikibot hosting departs from the now not-true statement ("In the near future (mid-2015), the WMF will move it's repositories from Gerrit to Phabricator"). That didn't happened.