Decommission the FlowReplies instrument
Closed, ResolvedPublic
Actions

Assigned To

Authored By

	phuedx
	May 23 2022, 1:23 PM

Description

According to the EventLogging Schema Audit spreadsheet, the FlowReplies schema (and associated instrument) should be decommissioned.

TODO

Remove the FlowReplies EventLoggingSchemas entry
- https://gerrit.wikimedia.org/r/822627
Remove all code that logs FlowReplies events
- https://gerrit.wikimedia.org/r/822628
Mark the legacy FlowReplies schema as inactive
- See https://meta.wikimedia.org/w/index.php?title=Schema_talk%3AFlowReplies&type=revision&diff=23687274&oldid=13302837
Request the deletion of any previously logged data, if necessary

T315857: Drop event.flowreplies table

Remove the FlowReplies event sanitisation allowlist entry
- https://gerrit.wikimedia.org/r/822630
Request the deletion of any previously sanitised data, if necessary

Notes

Per https://meta.wikimedia.org/w/index.php?title=Schema_talk:FlowReplies&action=history, the schema hasn't been updated since Aug. 2015
The Flow Reportcard Dashiki dashboard hasn't been updated since July 2019

Details

Subject	Repo	Branch	Lines +/-
Remove instrumentation	mediawiki/extensions/Flow	master	+10 -268
Remove FlowReplies EventLoggingSchemas entry	mediawiki/extensions/Flow	master	+0 -7
Remove FlowReplies allowlist entry	analytics/refinery	master	+0 -10

Customize query in gerrit

Related Objects
Search...

Status	Assigned	Task
Resolved	Ottomata	T185233 Modern Event Platform
Open	Ottomata	T238230 Decommission EventLogging backend components by migrating to MEP
Declined	None	T259734 BUOD-KR1-Q4+: Certify that analytics schema and instruments have been upgraded to use the MEP system (clearing the legacy system for sunsetting)
Open	Ottomata	T259163 Migrate legacy metawiki schemas to Event Platform
Resolved	Ottomata	T282131 Determine which remaining legacy EventLogging schemas need to be migrated or decommissioned
Open	None	T315857 Drop event.flowreplies table
Resolved	phuedx	T309025 Decommission the FlowReplies instrument

Event Timeline

phuedx created this task.May 23 2022, 1:23 PM

Restricted Application added a project: Growth-Team. · View Herald TranscriptMay 23 2022, 1:23 PM

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

phuedx updated the task description. (Show Details)May 23 2022, 1:29 PM

phuedx added a subscriber: • ppelberg.May 26 2022, 8:12 AM

• mewoph moved this task from Inbox to Needs Discussion on the Growth-Team board.May 26 2022, 10:27 PM

phuedx added a subscriber: MNeisler.Jun 15 2022, 10:47 AM

hi @phuedx – please consider this comment confirmation that the Editing Team is not using the FlowReplies instrumentation nor do we have plans to use it in the future.

As such, we (@MNeisler included) do not have concerns with y'all decommissioning this instrumentation.

MShilova_WMF moved this task from Needs Discussion to Triaged on the Growth-Team board.Aug 9 2022, 6:57 PM

Change 822627 had a related patch set uploaded (by Phuedx; author: Phuedx):

[mediawiki/extensions/Flow@master] Remove FlowReplies EventLoggingSchemas entry

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

Change 822628 had a related patch set uploaded (by Phuedx; author: Phuedx):

[mediawiki/extensions/Flow@master] Remove instrumentation

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

phuedx updated the task description. (Show Details)Aug 12 2022, 1:11 PM

Change 822630 had a related patch set uploaded (by Phuedx; author: Phuedx):

[analytics/refinery@master] Remove FlowReplies allowlist entry

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

phuedx updated the task description. (Show Details)Aug 12 2022, 1:26 PM

Change 822630 merged by Mforns:

[analytics/refinery@master] Remove FlowReplies allowlist entry

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

Change 822627 merged by jenkins-bot:

[mediawiki/extensions/Flow@master] Remove FlowReplies EventLoggingSchemas entry

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

Change 822628 merged by jenkins-bot:

[mediawiki/extensions/Flow@master] Remove instrumentation

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

Jdforrester-WMF updated the task description. (Show Details)Aug 19 2022, 2:48 PM

ReleaseTaggerBot added a project: MW-1.39-notes (1.39.0-wmf.26; 2022-08-22).Aug 19 2022, 3:00 PM

phuedx updated the task description. (Show Details)Aug 19 2022, 3:05 PM

phuedx updated the task description. (Show Details)

Maintenance_bot removed a project: Patch-For-Review.Aug 19 2022, 3:30 PM

phuedx mentioned this in T315857: Drop event.flowreplies table.Aug 22 2022, 9:26 AM

phuedx added a parent task: T315857: Drop event.flowreplies table.Aug 22 2022, 9:29 AM

Being bold. Thanks to @ppelberg for following up on whether the instrument should be decommissioned or migrated and to @Jdforrester-WMF for merging the patches.

Decommission the FlowReplies instrumentClosed, ResolvedPublicActions