Page MenuHomePhabricator

Remove action_context from ReadingLists AB test page_load event
Closed, ResolvedPublic2 Estimated Story Points

Description

Per @jwang, we do not need action_context for the page_load event in the ReadingLists AB test.

We should remove it from the event.

Follow up: We were recording the page title for the page_load event which is not needed for the experiment, and could elevate our data-collection risk tier.

Event Timeline

Change #1207218 had a related patch set uploaded (by Aude; author: Aude):

[mediawiki/extensions/WikimediaEvents@master] Remove action_context from page_load events in ReadingList A/B test

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

Change #1207218 merged by jenkins-bot:

[mediawiki/extensions/WikimediaEvents@master] Remove action_context from page_load events in ReadingList A/B test

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

Change #1207245 had a related patch set uploaded (by Aude; author: Aude):

[mediawiki/extensions/WikimediaEvents@wmf/1.46.0-wmf.2] Remove action_context from page_load events in ReadingList A/B test

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

Change #1207246 had a related patch set uploaded (by Aude; author: Aude):

[mediawiki/extensions/WikimediaEvents@wmf/1.46.0-wmf.3] Remove action_context from page_load events in ReadingList A/B test

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

Change #1207245 merged by jenkins-bot:

[mediawiki/extensions/WikimediaEvents@wmf/1.46.0-wmf.2] Remove action_context from page_load events in ReadingList A/B test

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

Change #1207246 merged by jenkins-bot:

[mediawiki/extensions/WikimediaEvents@wmf/1.46.0-wmf.3] Remove action_context from page_load events in ReadingList A/B test

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

Mentioned in SAL (#wikimedia-operations) [2025-11-19T22:05:55Z] <kemayo@deploy2002> Started scap sync-world: Backport for [[gerrit:1207201|TextMatchEditCheck: undo duplicate sub-type logging (T407286)]], [[gerrit:1207245|Remove action_context from page_load events in ReadingList A/B test (T410535)]], [[gerrit:1207246|Remove action_context from page_load events in ReadingList A/B test (T410535)]], [[gerrit:1207241|README: remove outdated advice about dblists]]

Mentioned in SAL (#wikimedia-operations) [2025-11-19T22:10:56Z] <kemayo@deploy2002> aude, kemayo, novemlinguae: Backport for [[gerrit:1207201|TextMatchEditCheck: undo duplicate sub-type logging (T407286)]], [[gerrit:1207245|Remove action_context from page_load events in ReadingList A/B test (T410535)]], [[gerrit:1207246|Remove action_context from page_load events in ReadingList A/B test (T410535)]], [[gerrit:1207241|README: remove outdated advice about dblists]] synced to the tests

Mentioned in SAL (#wikimedia-operations) [2025-11-19T22:16:52Z] <kemayo@deploy2002> Finished scap sync-world: Backport for [[gerrit:1207201|TextMatchEditCheck: undo duplicate sub-type logging (T407286)]], [[gerrit:1207245|Remove action_context from page_load events in ReadingList A/B test (T410535)]], [[gerrit:1207246|Remove action_context from page_load events in ReadingList A/B test (T410535)]], [[gerrit:1207241|README: remove outdated advice about dblists]] (duration: 10m 57s)

aude updated the task description. (Show Details)

This looks good to me on superset.

@jwang is this working as expected for you?

Confirmed with @jwang that this issue has been resolved.

Jdlrobson-WMF set the point value for this task to 2.Dec 2 2025, 5:10 PM