Maniphest T219713

CX: Explore the feasibility to reduce CI dependencies and time taken
Closed, ResolvedPublic
Actions

Description

Revisit the CI dependencies and explore any steps to reduce the time taken for CIs

Details

Subject	Repo	Branch	Lines +/-
Make the wikibase dependency conditional and soft	mediawiki/extensions/ContentTranslation	REL1_31	+12 -11
ContentTranslation: Restore Wikibase CI dependency	integration/config	master	+2 -1
Make the wikibase dependency conditional and soft	mediawiki/extensions/ContentTranslation	master	+12 -11
Remove the usage of guided tour	mediawiki/extensions/ContentTranslation	master	+0 -11
ContentTranslation: Remove Wikibase and GuidedTour CI dependency	integration/config	master	+2 -3
Remove CX1 after publish guided tour	mediawiki/extensions/ContentTranslation	master	+0 -259

Customize query in gerrit

Related Objects

Mentioned In: T263509: The module 'wikibase.api.RepoApi' required by 'ext.cx.wikibase.link' must exist

Event Timeline

• santhosh created this task.Mar 30 2019, 6:32 PM

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptMar 30 2019, 6:32 PM

Pginer-WMF edited projects, added Language-Team (Language-2019-April-June); removed Language-Team (Language-2019-January-March).Apr 2 2019, 11:26 AM

Current dependencies:

'ContentTranslation': ['AbuseFilter', 'Echo', 'EventLogging', 'GuidedTour',  'UniversalLanguageSelector', 'Wikibase',   'VisualEditor'],

Wikibase has the following dependencies:

'Wikibase': [
       'ArticlePlaceholder',
       'CirrusSearch',
       'cldr',
       'Elastica',
       'GeoData',
       'Scribunto',
       'Capiunto',
       'Echo',
       'PropertySuggester',
       'WikibaseQualityConstraints',
       'WikimediaBadges',
       'WikibaseMediaInfo',
       'WikibaseLexeme'
   ],

'AbuseFilter': ['AntiSpoof', 'CentralAuth', 'CodeEditor',  'CheckUser', 'Renameuser'],
'VisualEditor': ['Cite', 'TemplateData'],
'CirrusSearch': ['TimedMediaHandler', 'PdfHandler', 'Cite', 'Elastica',  'GeoData', 'BetaFeatures', 'SiteMatrix'],
 'Echo': ['CentralAuth', 'EducationProgram', 'EventLogging'],

and so on.

Our extension.json lists two requirements

	"VisualEditor": ">= 0.1.0",
	"UniversalLanguageSelector": "*"

GuidedTour is a soft dependency. We check whether it is installed or not. Perhaps that can be removed.
Wikibase client code to add interlanguage links is based on a hook handler. The modules is loaded only if Wikibase is present in the wiki- so no hard depdendency there too. I think it can be removed without much harm.

Change 500810 had a related patch set uploaded (by Santhosh; owner: Santhosh):
[integration/config@master] ContentTranslation: Remove Wikibase and GuidedTour CI dependency

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

gerritbot added a project: Patch-For-Review.Apr 2 2019, 6:53 PM

• santhosh triaged this task as Medium priority.Apr 2 2019, 7:07 PM

• santhosh moved this task from Backlog to In Review on the Language-Team (Language-2019-April-June) board.

Change 500811 had a related patch set uploaded (by Nikerabbit; owner: Nikerabbit):
[mediawiki/extensions/ContentTranslation@master] Remove CX1 after publish guided tour

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

Change 500811 merged by jenkins-bot:
[mediawiki/extensions/ContentTranslation@master] Remove CX1 after publish guided tour

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

ReleaseTaggerBot added a project: MW-1.33-notes (1.33.0-wmf.25; 2019-04-09).Apr 2 2019, 9:01 PM

Pginer-WMF moved this task from Needs Triage to Enhancements on the ContentTranslation board.Apr 8 2019, 8:56 AM

Change 500810 merged by jenkins-bot:
[integration/config@master] ContentTranslation: Remove Wikibase and GuidedTour CI dependency

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

Change 502560 had a related patch set uploaded (by Krinkle; owner: Krinkle):
[integration/config@master] ContentTranslation: Restore Wikibase CI dependency

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

Change 502560 merged by jenkins-bot:
[integration/config@master] ContentTranslation: Restore Wikibase CI dependency

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

Change 502671 had a related patch set uploaded (by Santhosh; owner: Santhosh):
[mediawiki/extensions/ContentTranslation@master] Remove the usage of guided tour

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

Change 502672 had a related patch set uploaded (by Santhosh; owner: Santhosh):
[mediawiki/extensions/ContentTranslation@master] Make the wikibase dependency conditional and soft

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

Change 502671 merged by jenkins-bot:
[mediawiki/extensions/ContentTranslation@master] Remove the usage of guided tour

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

ReleaseTaggerBot added a project: MW-1.34-notes (1.34.0-wmf.1; 2019-04-16).Apr 10 2019, 7:01 AM

Change 502672 merged by jenkins-bot:
[mediawiki/extensions/ContentTranslation@master] Make the wikibase dependency conditional and soft

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

With the above fixes merged, we can remove the Wikibase dependency again.

• santhosh moved this task from In Review to Done on the Language-Team (Language-2019-April-June) board.Apr 12 2019, 2:58 AM

Pginer-WMF closed this task as Resolved.Apr 23 2019, 8:36 AM

Pginer-WMF claimed this task.

Reedy mentioned this in T263509: The module 'wikibase.api.RepoApi' required by 'ext.cx.wikibase.link' must exist.Sep 22 2020, 12:47 AM

Change 628993 had a related patch set uploaded (by Reedy; owner: Santhosh):
[mediawiki/extensions/ContentTranslation@REL1_31] Make the wikibase dependency conditional and soft

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

gerritbot added a project: Patch-For-Review.Sep 22 2020, 12:48 AM

Change 628993 merged by jenkins-bot:
[mediawiki/extensions/ContentTranslation@REL1_31] Make the wikibase dependency conditional and soft