Suggested investigations: '.performer.registration_dt' should be string
Closed, ResolvedPublic
Actions

Description

Summary

'.performer.registration_dt' should be string appears for several events in Logstash https://logstash.wikimedia.org/goto/a65f3e4ab64aa1c470f423fa3b426f50

Acceptance criteria

performer.registration_dt is not false for SI events

Details

Related Changes in Gerrit:

Subject	Repo	Branch	Lines +/-
Metrics Platform PHP client library: performer_registration_dt won't be added to the user when the user is anon	mediawiki/extensions/EventLogging	wmf/1.45.0-wmf.24	+9 -8
Metrics Platform PHP client library: performer_registration_dt won't be added to the user when the user is anon	mediawiki/extensions/EventLogging	wmf/1.45.0-wmf.25	+9 -8
Metrics Platform PHP client library: set performer_registration_dt as null when the user is anon	mediawiki/extensions/EventLogging	master	+9 -8

Customize query in gerrit

Event Timeline

kostajh created this task.Oct 28 2025, 1:03 PM

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptOct 28 2025, 1:03 PM

kostajh added a project: Metrics Platform.Oct 28 2025, 1:39 PM

Sfaci added a project: Test Kitchen (Experiment Platform Sprint 14).Oct 28 2025, 1:43 PM

Sfaci moved this task from Ready for Development to In Progress on the Test Kitchen (Experiment Platform Sprint 14) board.

performer_registration_dt is a contextual attributes that must be filled by the PHP metrics client library in this case

It seems that PHP metrics platform client library is setting this attribute as false when user->isAnon here and nothing else happens for that case (if the user is registered the right date is stored there). I guess the value should be set as null to avoid validation errors when being validated for the eventgate

We'll prepare a patch to fix it

Sfaci claimed this task.Oct 28 2025, 1:48 PM

OKryva-WMF moved this task from Backlog to Ready on the Product Safety and Integrity (Sprint Mint Choc Chip Ice Cream (Oct 20 - Nov 7)) board.Oct 28 2025, 1:50 PM

OKryva-WMF moved this task from Backlog to Ready on the CheckUser-SuggestedInvestigations board.Oct 28 2025, 2:59 PM

Change #1199293 had a related patch set uploaded (by Santiago Faci; author: Santiago Faci):

[mediawiki/extensions/EventLogging@master] Metrics Platform PHP client library: set performer_registration_dt as null when the user is anon

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

gerritbot added a project: Patch-For-Review.Oct 28 2025, 9:40 PM

Sfaci moved this task from In Progress to Needs Review on the Test Kitchen (Experiment Platform Sprint 14) board.Oct 28 2025, 9:40 PM

I guess the value should be set as null to avoid validation errors when being validated for the eventgate

I said that but @mpopov wisely suggested to avoid adding the field instead of setting as null. The validation would also fail for null values. The patch above makes PHP client library not add the performer_registration_dt to the event is the user is not registered.

Change #1199293 merged by jenkins-bot:

[mediawiki/extensions/EventLogging@master] Metrics Platform PHP client library: set performer_registration_dt as null when the user is anon

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

ReleaseTaggerBot added a project: MW-1.46-notes (1.46.0-wmf.1; 2025-11-05).Oct 28 2025, 11:01 PM

Maintenance_bot removed a project: Patch-For-Review.Oct 28 2025, 11:33 PM

Change #1199524 had a related patch set uploaded (by Santiago Faci; author: Santiago Faci):

[mediawiki/extensions/EventLogging@wmf/1.45.0-wmf.25] Metrics Platform PHP client library: set performer_registration_dt as null when the user is anon

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

Change #1199525 had a related patch set uploaded (by Santiago Faci; author: Santiago Faci):

[mediawiki/extensions/EventLogging@wmf/1.45.0-wmf.24] Metrics Platform PHP client library: set performer_registration_dt as null when the user is anon

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

Sfaci moved this task from Needs Review to To Deploy on the Test Kitchen (Experiment Platform Sprint 14) board.Oct 29 2025, 7:00 AM

Change #1199524 merged by jenkins-bot:

[mediawiki/extensions/EventLogging@wmf/1.45.0-wmf.25] Metrics Platform PHP client library: performer_registration_dt won't be added to the user when the user is anon

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

Change #1199525 merged by jenkins-bot:

[mediawiki/extensions/EventLogging@wmf/1.45.0-wmf.24] Metrics Platform PHP client library: performer_registration_dt won't be added to the user when the user is anon

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

Mentioned in SAL (#wikimedia-operations) [2025-10-29T08:35:34Z] <kharlan@deploy2002> Started scap sync-world: Backport for [[gerrit:1199524|Metrics Platform PHP client library: performer_registration_dt won't be added to the user when the user is anon (T408547)]], [[gerrit:1199525|Metrics Platform PHP client library: performer_registration_dt won't be added to the user when the user is anon (T408547)]]

Mentioned in SAL (#wikimedia-operations) [2025-10-29T08:38:05Z] <kharlan@deploy2002> sfaci, kharlan: Backport for [[gerrit:1199524|Metrics Platform PHP client library: performer_registration_dt won't be added to the user when the user is anon (T408547)]], [[gerrit:1199525|Metrics Platform PHP client library: performer_registration_dt won't be added to the user when the user is anon (T408547)]] synced to the testservers (see https://wikitech.wikimedia.org/wiki/Mwdebug). Changes can

Mentioned in SAL (#wikimedia-operations) [2025-10-29T08:44:21Z] <kharlan@deploy2002> Finished scap sync-world: Backport for [[gerrit:1199524|Metrics Platform PHP client library: performer_registration_dt won't be added to the user when the user is anon (T408547)]], [[gerrit:1199525|Metrics Platform PHP client library: performer_registration_dt won't be added to the user when the user is anon (T408547)]] (duration: 08m 47s)