Migrate Differential repo rWIEG back to Gerrit wikimedia/iegreview.git
Closed, ResolvedPublic
Actions

Assigned To

Authored By

	bd808
	Nov 30 2018, 3:32 AM

Description

Basically undo T132174: Migrate wikimedia-iegreview to Differential.

Wikimedia-IEG-grant-review was an early adopter of Differential in anticipation of all Wikimedia projects migrating to it. Now that the use of Differential has been ruled out as a global goal (T119908#4033567) we should return to Gerrit.

Details

	Subject	Repo	Branch	Lines +/-
	Revert "Remove .gitreview"	wikimedia/iegreview	master	+5 -24
	Revert "Remove wikimedia/iegreview (moved to Phabricator)"	integration/config	master	+4 -0

Customize query in gerrit

Related Objects
Search...

Status	Assigned	Task
Open	None	T330844 Avoid to have too much different places (like Gerrit, GitHub, Differential, GitLab, ...) where code review may take place to help newcomers
Resolved	Aklapper	T342041 Remove custom src/customfields/DifferentialApplyPatchWithOnlyGitField.php
Resolved	Aklapper	T330797 Uninstall Differential (Phabricator application)
Resolved	Aklapper	T324131 Remove "Active Differential users" stats from monthly community_metrics.sh email
Resolved	Aklapper	T191182 Migrate active repositories in Phabricator Differential to GitLab
Resolved	Legoktm	T210820 Migrate Differential repo rWIEG back to Gerrit wikimedia/iegreview.git

Event Timeline

bd808 created this task.Nov 30 2018, 3:32 AM

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptNov 30 2018, 3:33 AM

hashar added a parent task: T191182: Migrate active repositories in Phabricator Differential to GitLab.Nov 30 2018, 8:21 AM

Paladox moved this task from Bugs & stuff to Repo Admin on the Gerrit board.Dec 14 2018, 9:11 PM

greg added a project: Diffusion-Repository-Administrators.Dec 20 2018, 4:35 AM

Change 480907 had a related patch set uploaded (by Legoktm; owner: Legoktm):
[wikimedia/iegreview@master] Revert "Remove .gitreview"

https://gerrit.wikimedia.org/r/480907

Change 480908 had a related patch set uploaded (by Legoktm; owner: Legoktm):
[integration/config@master] Revert "Remove wikimedia/iegreview (moved to Phabricator)"

https://gerrit.wikimedia.org/r/480908

Change 480908 merged by jenkins-bot:
[integration/config@master] Revert "Remove wikimedia/iegreview (moved to Phabricator)"

https://gerrit.wikimedia.org/r/480908

I re-activated wikimedia/iegreview.git on Gerrit, and swapped Phabricator to mirror from it.

And re-enabled CI, except the PHP 7.0 tests are failing so patches can't be merged. I've left the tests enabled for now, please file a bug in Continuous-Integration-Config if the configuration needs adjustment.

21:17:22 1) Wikimedia\IEGReview\ArrayTest::testDifferenceException with data set #0 ('Lorem Ipsum', 'Ipsum Test')
21:17:22 Failed asserting that exception of type "TypeError" matches expected exception "PHPUnit_Framework_Error".
21:17:22 
21:17:22 2) Wikimedia\IEGReview\ArrayTest::testDifferenceException with data set #1 ('', '')
21:17:22 Failed asserting that exception of type "TypeError" matches expected exception "PHPUnit_Framework_Error".

Legoktm mentioned this in rWIEG0f9988973ba3: Revert "Remove .gitreview".Dec 20 2018, 5:47 AM

Change 480907 merged by jenkins-bot:
[wikimedia/iegreview@master] Revert "Remove .gitreview"

https://gerrit.wikimedia.org/r/480907

bd808 mentioned this in rWIEG9bc6f8af6796: Revert "Remove .gitreview".Dec 27 2018, 7:34 PM

bd808 mentioned this in rWIEG52eb4e72ef5e: Revert "Remove .gitreview".