rTLDAP tool-ldap continues to exist in Differential. Let's move it to gerrit.
Description
Details
Subject | Repo | Branch | Lines +/- | |
---|---|---|---|---|
Add .gitreview | labs/tools/ldap | master | +6 -0 |
Status | Subtype | 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 | MarcoAurelio | T219703 Move `tool-ldap` from Differential to Gerrit |
Event Timeline
This is basically prompted due to the complexity of Differential to submit patches for review, specially when this tool doesn't have any .arcconfig or .gitreview file. To make {D1146} (pending review fwiw) I had to go here and there and upload a raw patch to the UI, etc. I don't mind keeping a mirror on Diffusion of course, but development should be switched either to Gerrit or to GitHub. Thank you.
Please. It would be nice if all of my tools could be moved en-masse. I suppose it would be worthwhile to script it?
I can migrate (import) the repo contents over to Gerrit but then you'll have to change the clone links so they point to the actuall gerrit repo instead of the diffusion one, which will remain as a mirror of the GitHub one.
Mentioned in SAL (#wikimedia-releng) [2019-04-01T21:33:40Z] <hauskatze> Created https://gerrit.wikimedia.org/r/#/admin/projects/labs/tools/ldap | T219703
Mentioned in SAL (#wikimedia-releng) [2019-04-01T21:42:23Z] <hauskatze> Imported tool-ldap from Diffusion to Gerrit with full history | T219703
Change 500617 had a related patch set uploaded (by MarcoAurelio; owner: MarcoAurelio):
[labs/tools/ldap@master] Add .gitreview
@Legoktm The tool complete history, commits, etc. has been imported to Gerrit (labs/tools/ldap, per naming scheme). I guess you'll need to make some changes in the tool to pick the repo from Gerrit instead of Diffusion now? Once done, I can turn the Diffusion repo into a mirror of our Gerrit one.
tools.ldap@tools-sgebastion-07:~/www/python/src$ git remote -v origin https://phabricator.wikimedia.org/source/tool-ldap.git (fetch) origin https://phabricator.wikimedia.org/source/tool-ldap.git (push) tools.ldap@tools-sgebastion-07:~/www/python/src$ git remote set-url origin https://gerrit.wikimedia.org/r/labs/tools/ldap tools.ldap@tools-sgebastion-07:~/www/python/src$ git remote -v origin https://gerrit.wikimedia.org/r/labs/tools/ldap (fetch) origin https://gerrit.wikimedia.org/r/labs/tools/ldap (push) tools.ldap@tools-sgebastion-07:~/www/python/src$ git pull remote: Counting objects: 4, done remote: Finding sources: 100% (3/3) remote: Getting sizes: 100% (3/3) remote: Compressing objects: 100% (290/290) remote: Total 3 (delta 1), reused 0 (delta 0) Unpacking objects: 100% (3/3), done. From https://gerrit.wikimedia.org/r/labs/tools/ldap d797c2b..30de8d5 master -> origin/master Updating d797c2b..30de8d5 Fast-forward .gitreview | 6 ++++++ 1 file changed, 6 insertions(+) create mode 100644 .gitreview
Should be fine to switch over.
Syncronization between Gerrit <-> Diffusion seems to work: 30de8d55 was just imported.
I consider this task as resolved.