Page MenuHomePhabricator

cleanMentorList is unable to remove locked mentors from mentorship
Open, LowPublic

Description

See this from dewiki (script execution happened for T431804):

[urbanecm@deploy1003 ~]$ mwscript-k8s -f GrowthExperiments:cleanMentorList.php -- --wiki=dewiki
⏳ Starting GrowthExperiments:cleanMentorList.php on Kubernetes as job mw-script.eqiad.i98y0pye ...
⏳ Waiting for the container to start...
🚀 Job is running.
📜 Streaming logs:
Renamed user 2e233d074791c0e6ac153f38829154f0 {{GENDER:Renamed user 2e233d074791c0e6ac153f38829154f0|is}} currently locked. {{GENDER:Renamed user 2e233d074791c0e6ac153f38829154f0|Their}} mentor status cannot be modified while locked.[urbanecm@deploy1003 ~]$

cleanMentorList should definitely be able to remove mentors who are locked. Not sure what part stopped this from happening.

T433243: Vanishing a mentor should automatically remove them from mentorship is a similar task that would remove mentors automatically on vanishing. It is a distinct task from this one.