Add Github plugin to Gerrit to support 2 way sync
Closed, DeclinedPublic
Actions

Assigned To

Authored By

	• Fjalapeno
	Mar 14 2015, 1:40 AM

Description

The Apps team would like to be able to accept pull requests on Github from the community.

To do this we need to a way to convert pull requests into patch sets on Gerrit.

This functionality (I believe) previously existed, but is no longer working and/or supported.

After an internet search, we discovered the following plugin:
https://gerrit.googlesource.com/plugins/github/

This looks like a good way to support this activity.

Could you let us know if it possible for us to install this plug in, or if not other options to support the 2 way syncing behavior?

Thanks!

Related Objects

Mentioned Here: T560: Proof of concept of code review in Phabricator
T18: Plan to migrate code review from Gerrit to Phabricator

Event Timeline

• Fjalapeno created this task.Mar 14 2015, 1:40 AM

• Fjalapeno raised the priority of this task from to Needs Triage.

• Fjalapeno updated the task description. (Show Details)

• Fjalapeno added subscribers: • Fjalapeno, • BGerstle-WMF, • bearND, • KLans_WMF.

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptMar 14 2015, 1:40 AM

• Fjalapeno added a project: Gerrit.Mar 14 2015, 1:40 AM

• Fjalapeno set Security to None.

Krenair subscribed.Mar 14 2015, 1:42 AM

It seems that WMF is sold on moving code-review to phabricator and not update our gerrit.

Our gerrit is currently at an old 2.8 version.
And the corresponding 2.8 versions of the github plugin did not work as expected when I last tested them.

Hence, declining :-(

@QChris do you happen to have a link to a doc which provides more background info on this issue? Also, I've heard various estimates on how long it will be until Diffusion is ready for mainstream use (~6 months). Further, you mentioned that Diffusion won't support build server integrations (e.g. Jenkins). Is there any way we can make a reasonable investment to improve gerrit while we wait for Diffusion to reach parity?

• Fjalapeno added a subscriber: dr0ptp4kt.Mar 16 2015, 4:19 PM

In T92706#1121697, @BGerstle-WMF wrote:

@QChris do you happen to have a link to a doc which provides more background info on this issue? [...]

The relevant bug seems to be T18 (That bug also has a rough timeline).

The main decision to move from gerrit seems to stem from https://www.mediawiki.org/wiki/Requests_for_comment/Phabricator and the discussions around it.

If you need more information about the move, I guess @demon would know more.

Also, I've heard various estimates on how long it will be until Diffusion is ready for mainstream use (~6 months).

No clue here. T18 has a rough timeline, which points to June the latest.

Further, you mentioned that Diffusion won't support build server integrations (e.g. Jenkins). Is there any way we can make a reasonable investment to improve gerrit while we wait for Diffusion to reach parity?

I think that's a question for WMF people to answer. Up to now, I only heard that WMF will not invest in Gerrit further. @demon would know better.

General timeline: https://www.mediawiki.org/wiki/Phabricator#Migration_timeline - some dates might be too optimistic. Dates in T18 are outdated. T560 is the main next task to concentrate on when it comes to Gerrit/Differential. (Diffusion is the repository browser in Phabricator).

Add Github plugin to Gerrit to support 2 way syncClosed, DeclinedPublicActions

Description

Related Objects

Event Timeline

Add Github plugin to Gerrit to support 2 way sync
Closed, DeclinedPublic
Actions