Change 713072 had a related patch set uploaded (by Zabe; author: Zabe):

[mediawiki/extensions/CentralAuth@master] maintenance: fix require_once in migrateStewards.php

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

Patch merged, but I wouldn't be surprised if the script just continued to not work even with it -- the extension was designed for Wikimedia's purposes, and no one probably run it in a very long time.

Change 713072 merged by jenkins-bot:

[mediawiki/extensions/CentralAuth@master] maintenance: fix require_once in migrateStewards.php

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

In T288903#7283496, @Urbanecm wrote:

Patch merged, but I wouldn't be surprised if the script just continued to not work even with it -- the extension was designed for Wikimedia's purposes, and no one probably run it in a very long time.

CA isn't supported outside Wikimedia wikis and this script isn't really useful to us anymore. I kind of have hopes of some other migration scripts being useful if we ever make Wikitech a SUL wiki, but this is not one of them and could possibly just be removed.

In T288903#7283496, @Urbanecm wrote:

Patch merged, but I wouldn't be surprised if the script just continued to not work even with it -- the extension was designed for Wikimedia's purposes, and no one probably run it in a very long time.

Yes, the issue must have been there for many months and no one noticed, so the script is pretty much unused.

In T288903#7283499, @Majavah wrote:

In T288903#7283496, @Urbanecm wrote:

Patch merged, but I wouldn't be surprised if the script just continued to not work even with it -- the extension was designed for Wikimedia's purposes, and no one probably run it in a very long time.

CA isn't supported outside Wikimedia wikis and this script isn't really useful to us anymore. I kind of have hopes of some other migration scripts being useful if we ever make Wikitech a SUL wiki, but this is not one of them and could possibly just be removed.

I'm fine with removing it.

In T288903#7283499, @Majavah wrote:

In T288903#7283496, @Urbanecm wrote:

Patch merged, but I wouldn't be surprised if the script just continued to not work even with it -- the extension was designed for Wikimedia's purposes, and no one probably run it in a very long time.

CA isn't supported outside Wikimedia wikis and this script isn't really useful to us anymore. I kind of have hopes of some other migration scripts being useful if we ever make Wikitech a SUL wiki, but this is not one of them and could possibly just be removed.

I'd agree with removing it.

TBH, I doubt Wikitech will become a SUL wiki in the foreseeable future (it has many consequences/blockers -- to name some, move any kind of LDAP relevance out of wikitech). It will hopefully become hosted together with all the other wikis soon though :-).

In T288903#7283504, @Urbanecm wrote:

TBH, I doubt Wikitech will become a SUL wiki in the foreseeable future (it has many consequences/blockers -- to name some, move any kind of LDAP relevance out of wikitech). It will hopefully become hosted together with all the other wikis soon though :-).

True, but one can still hope, right? :P

How would you feel about removing most of the SUL migration code from CA and rewriting it if Wikitech SUL migration ever becomes realistic in the short term? I'd be surprised if any large parts of it functioned properly these days without a major rewrite and doing so would eliminate the maintenance burden of that code until/unless it would be needed.

In T288903#7283505, @Majavah wrote:

[...]
How would you feel about removing most of the SUL migration code from CA and rewriting it if Wikitech SUL migration ever becomes realistic in the short term? I'd be surprised if any large parts of it functioned properly these days without a major rewrite and doing so would eliminate the maintenance burden of that code until/unless it would be needed.

I'd feel that should be done, but...I also think that should have a separate task, so others can chime in if they want to.

In T288903#7283506, @Urbanecm wrote:

I'd feel that should be done, but...I also think that should have a separate task, so others can chime in if they want to.

T288906: Remove CentralAuth SUL migration code