Remove wfIncrStats logging from Special:MobileOptions
Closed, ResolvedPublic1 Estimated Story Points
Actions

Description

Context

During T122504, I found that we log MobileOptionsTracking events whenever a user updates their mobile preferences and asked Readers-Web-Backlog whether such logging was valuable, to which the answer was "Yes." @Jdlrobson noted that we also log mobile preference updates with calls to wfIncrStats but this should have been superseded by the aforementioned event logging.

AC

All calls to wfIncrStats are removed from the Special:MobileOptions code
An email is sent to wikitech-l, mobile-l, and reading-wmf with details of the forthcoming change

TBD

The statistics gathered via wfIncrStats are purged

Details

	Project	Branch	Lines +/-	Subject
	mediawiki/extensions/MobileFrontend	master	+0 -3	Remove wfIncrStats logging from Special:MobileOptions

Customize query in gerrit

Related Objects

Mentioned Here: T122504: Refactor mobile schemas

Event Timeline

phuedx created this task.Apr 8 2016, 8:40 AM

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptApr 8 2016, 8:40 AM

• jhobs triaged this task as Medium priority.Apr 12 2016, 5:15 PM

• jhobs added a project: Technical-Debt.

MBinder_WMF added a project: Readers-Web-Backlog.Apr 27 2016, 4:18 PM

Restricted Application added a subscriber: TerraCodes. · View Herald TranscriptApr 27 2016, 4:18 PM

Danny_B moved this task from Unsorted to Needs removal on the Technical-Debt board.May 27 2016, 11:01 PM

Jhernandez moved this task from Incoming to Triaged but Future on the Readers-Web-Backlog board.Jul 27 2016, 4:47 PM

MBinder_WMF added a project: Reading-Web-Tech-Debt.Oct 3 2016, 8:34 PM

MBinder_WMF edited projects, added Technical-Debt (RW-Tech-Debt); removed Technical-Debt.Oct 4 2016, 6:15 PM

MBinder_WMF removed a project: Reading-Web-Tech-Debt.Oct 4 2016, 7:53 PM

bmansurov moved this task from Uncategorized to Clean-Up on the Technical-Debt (RW-Tech-Debt) board.Oct 5 2016, 10:44 PM

A part of this task has been imported as a Code In task at https://codein.withgoogle.com/dashboard/tasks/6685801721102336/

Florian moved this task from Proposed tasks to Imported in GCI Site on the Google-Code-In-2016 board.Jan 3 2017, 5:26 PM

I'll clean this up, since GCI is almost over. :)

Change 332085 had a related patch set uploaded (by MtDu):
Remove wfIncrStats logging from Special:MobileOptions

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

gerritbot added a project: Patch-For-Review.Jan 14 2017, 11:18 PM

Note: There are also some calls in includes/MobileContext.php and includes/api/ApiMobileView.php. I'm not sure if we should remove those as well.

In T132134#2940619, @MtDu wrote:

Note: There are also some calls in includes/MobileContext.php and includes/api/ApiMobileView.php. I'm not sure if we should remove those as well.

Thanks for the reminder! Those calls are out of scope for this task.

phuedx added a project: Google-Code-In-2016.Jan 15 2017, 7:57 AM

phuedx updated the task description. (Show Details)

Change 332085 merged by jenkins-bot:
Remove wfIncrStats logging from Special:MobileOptions

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

ReleaseTaggerBot added a project: MW-1.29-release (WMF-deploy-2017-01-17_(1.29.0-wmf.8)).Jan 15 2017, 9:00 AM

@phuedx can you sign off?

Jdlrobson moved this task from Needs Analysis to Ready for Signoff on the Reading-Web-Sprint-90-🍌 board.Jan 24 2017, 7:41 PM

@Jdlrobson: Not until the email (AC #2) is sent. The email should probably be sent to analytics too and ask how the data gathered via wfIncrStats can be purged.

phuedx set the point value for this task to 1.Jan 25 2017, 4:28 PM

Here's a link to the mobile-l thread: https://lists.wikimedia.org/pipermail/mobile-l/2017-January/010445.html

phuedx updated the task description. (Show Details)Jan 25 2017, 8:23 PM

I've pinged Analytics asking how we go about getting stats purged from statsd along with the relevant keys.

We can reopen this task if purging the stats from statsd is non-trivial.

MBinder_WMF moved this task from Triaged but Future to 2016-17 Q3 on the Readers-Web-Backlog board.Oct 10 2017, 7:38 PM

Remove wfIncrStats logging from Special:MobileOptionsClosed, ResolvedPublic1 Estimated Story PointsActions