Maniphest T373203

MessageProcessor produces no logs
Closed, ResolvedPublicBUG REPORT
Actions

Assigned To

Authored By

	Urbanecm_WMF
	Aug 23 2024, 2:29 PM

Tags

Referenced Files

None

Subscribers

Description

While debugging T373202: MessageProcessor is fetching non-lowercase messages for enums, I found out that while MessageProcessor is issuing calls to a logger, it sends all its logs to a NullLogger, since we do not inject a valid logger to that service. This results in log entries being possibly discarded.

Details

	Subject	Repo	Branch	Lines +/-
	MessageProcessor: Actually log something	mediawiki/extensions/CommunityConfiguration	master	+4 -1

Customize query in gerrit

Related Objects

Mentioned In: rECOMda5e853a6916: MessageProcessor: Actually log something
Mentioned Here: T373202: MessageProcessor is fetching non-lowercase messages for enums

Event Timeline

Urbanecm_WMF created this task.Aug 23 2024, 2:29 PM

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptAug 23 2024, 2:29 PM

Urbanecm_WMF triaged this task as High priority.Aug 23 2024, 2:29 PM

Urbanecm_WMF changed the subtype of this task from "Task" to "Bug Report".

Change #1065232 had a related patch set uploaded (by Urbanecm; author: Urbanecm):

[mediawiki/extensions/CommunityConfiguration@master] MessageProcessor: Improve logging

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

gerritbot added a project: Patch-For-Review.Aug 23 2024, 2:34 PM

Urbanecm_WMF moved this task from Incoming to Code Review on the Growth-Team (FY2024-25 Q1 Sprint 4) board.Aug 23 2024, 2:34 PM

Change #1065232 merged by jenkins-bot:

[mediawiki/extensions/CommunityConfiguration@master] MessageProcessor: Actually log something

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

Urbanecm mentioned this in rECOMda5e853a6916: MessageProcessor: Actually log something.Aug 28 2024, 8:12 AM

ReleaseTaggerBot added a project: MW-1.43-notes (1.43.0-wmf.21; 2024-09-03).Aug 28 2024, 9:00 AM

Urbanecm_WMF moved this task from Code Review to QA on the Growth-Team (FY2024-25 Q1 Sprint 4) board.Aug 28 2024, 3:45 PM

Urbanecm_WMF edited projects, added Growth-Team (FY2024-25 Q1 Sprint 5); removed Growth-Team (FY2024-25 Q1 Sprint 4).Sep 5 2024, 9:35 AM

Urbanecm_WMF moved this task from Incoming to QA on the Growth-Team (FY2024-25 Q1 Sprint 5) board.

Maintenance_bot removed a project: Patch-For-Review.Sep 5 2024, 10:31 AM

KStoller-WMF edited projects, added Growth-Team (FY2024-25 Q1 Sprint 6); removed Growth-Team (FY2024-25 Q1 Sprint 5).Sep 16 2024, 10:43 PM

KStoller-WMF moved this task from Incoming to QA on the Growth-Team (FY2024-25 Q1 Sprint 6) board.Sep 16 2024, 10:49 PM

Etonkovidova closed this task as Resolved.Sep 24 2024, 11:19 PM