ProofreadPages CI breaks without VE integration
Closed, ResolvedPublicBUG REPORT
Actions

Assigned To

Authored By

	ihurbain
	Nov 28 2025, 11:02 AM

Description

This empty patch https://gerrit.wikimedia.org/r/c/mediawiki/extensions/ProofreadPage/+/1212546 is not passing CI.

Current assumption: there is now a conditional load of VE (via https://gerrit.wikimedia.org/r/c/mediawiki/extensions/ProofreadPage/+/1193456), and I'm assuming that ParserFunctions would come as a dependency somewhere of that (probably via Cite), and if it's not loaded, the corresponding parserTests that use this extension are also broken.

MW_ZUUL_RECURSE is true, and EXT_DEPENDENCIES is

mediawiki/extensions/BetaFeatures
mediawiki/extensions/Cite
mediawiki/extensions/ConfirmEdit
mediawiki/extensions/DiscussionTools
mediawiki/extensions/FlaggedRevs
mediawiki/extensions/LabeledSectionTransclusion
mediawiki/extensions/Linter
mediawiki/extensions/Scribunto
mediawiki/extensions/TemplateData
mediawiki/extensions/VisualEditor

Details

Related Changes in Gerrit:

	Subject	Repo	Branch	Lines +/-
	Zuul: [ProofreadPage]: disable recursion	integration/config	master	+1 -1
	Zuul: [mediawiki/extensions/ProofreadPages] add ParserFunctions	integration/config	master	+2 -0

Customize query in gerrit

Related Objects
Search...

Status	Subtype	Assigned	Task
Open		None	T225730 Reduce runtime of MW shared gate Jenkins jobs to 5 min
Open		None	T391008 Reconcile dependencies.yaml and phan_dependencies.yaml
Open		hashar	T389998 Allow control over which extra extensions are installed (Math REL1_43 jobs exceed 60min timeout)
Resolved	BUG REPORT	hashar	T411246 ProofreadPages CI breaks without VE integration

Event Timeline

ihurbain created this task.Nov 28 2025, 11:02 AM

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptNov 28 2025, 11:02 AM

A_smart_kitten edited projects, added ci-test-error (WMF-deployed Build Failure), ProofreadPage; removed ci-test-error.Nov 28 2025, 1:42 PM

hashar updated the task description. (Show Details)Nov 28 2025, 3:29 PM

Change #1212601 had a related patch set uploaded (by Hashar; author: Hashar):

[integration/config@master] Zuul: [mediawiki/extensions/ProofreadPages] add ParserFunctions

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

gerritbot added a project: Patch-For-Review.Nov 28 2025, 3:33 PM

Indeed I broke it when I disabled recursion on VisualEditor ( https://gerrit.wikimedia.org/r/c/integration/config/+/1211189 ) which removed a lot of dependencies. Previously the long list included ParserFunctions and was:

AbuseFilter
AntiSpoof
BetaFeatures
CentralAuth
CheckUser
CirrusSearch
Cite
cldr
CodeEditor
CommunityConfiguration
CommunityConfigurationExample
ConfirmEdit
DiscussionTools
Echo
Elastica
EmailAuth
EventBus
EventLogging
EventStreamConfig
FlaggedRevs
Flow
Gadgets
GeoData
GlobalBlocking
GlobalPreferences
GrowthExperiments
GuidedTour
IPInfo
IPReputation
LabeledSectionTransclusion
Linter
LoginNotify
MetricsPlatform
MobileApp
MobileFrontend
OATHAuth
PageImages
PageViewInfo
ParserFunctions
PdfHandler
Popups
Renameuser
Scribunto
SecurePoll
SiteMatrix
TemplateData
TextExtracts
Thanks
TimedMediaHandler
TorBlock
VisualEditor
WebAuthn
WikiEditor
WikimediaEvents
WikimediaMessages

Change #1212601 merged by jenkins-bot:

[integration/config@master] Zuul: [mediawiki/extensions/ProofreadPages] add ParserFunctions

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