Page MenuHomePhabricator

Scaling: No mentor available for an user is treated as an error condition
Closed, ResolvedPublic

Description

Right now, MentorPageManager, the class that manages mentor-mentee relationship, treats "no mentor available" as an error, and logs it to logstash. As we want to be able to be able to deploy GrowthExperiments without mentorship module (T273776: Scale: ability to deploy without mentorship module), we should stop treating "no mentors available" scenario as an error condition, avoiding logspam on massive deployment.

This should be quite easy to implement.

Event Timeline

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

Change 664858 had a related patch set uploaded (by Urbanecm; owner: Urbanecm):
[mediawiki/extensions/GrowthExperiments@master] MentorPageMentorManager: Do not log "no mentor available" as warning

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

Change 664858 merged by jenkins-bot:
[mediawiki/extensions/GrowthExperiments@master] MentorPageMentorManager: Do not log "no mentor available" as warning

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

No way to QA this, closing right away.