backfilLocalAccounts.php should ignore warnings
Closed, ResolvedPublic
Actions

Assigned To

Authored By

	• Tgr
	Jul 20 2026, 3:41 PM

Project Tags

Referenced Files

None

Subscribers

Aklapper

Rajveer42

• Tgr

Description

AuthManager::autoCreateAccount() will return a successful status with a warning if the account already exists. backfilLocalAccounts.php checks the status with isGood() and treats this as an error. It should treat it as a success since there's nothing left to backfill.

Details

Related Changes in Gerrit:

	Subject	Author	Repo	Branch	Lines +/-
	backfillLocalAccounts: do not treat warnings as errors	Rajveer42	mediawiki/extensions/CentralAuth	master	+16 -1

Customize query in gerrit

Related Objects

Mentioned In: T431773: MediaWiki periodic job centralauth-backfilllocalaccounts.php-loginwiki failed

Event Timeline

• Tgr created this task.Jul 20 2026, 3:41 PM

Restricted Application added a project: MediaWiki-Core-Platform-Team. · View Herald TranscriptJul 20 2026, 3:41 PM

Restricted Application added a subscriber: Aklapper. · View Herald Transcript

• Tgr mentioned this in T431773: MediaWiki periodic job centralauth-backfilllocalaccounts.php-loginwiki failed.Jul 20 2026, 3:45 PM

I'd like to take this one, unless someone is already on it.

Rajveer42 claimed this task.Sun, Jul 26, 9:00 AM

Zabe renamed this task from backfilLocalAccounts.php shoudl ignore warnings to backfilLocalAccounts.php should ignore warnings.Sun, Jul 26, 9:48 AM

Change #1317601 had a related patch set uploaded (by Rajveer42; author: Rajveer42):

[mediawiki/extensions/CentralAuth@master] backfillLocalAccounts: do not treat warnings as errors

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

gerritbot added a project: Patch-For-Review.Sun, Jul 26, 12:26 PM

Thanks!

Change #1317601 merged by jenkins-bot: