Page MenuHomePhabricator

VirtualPageView Event Platform Migration
Closed, ResolvedPublic

Description

See: https://wikitech.wikimedia.org/wiki/Event_Platform/EventLogging_legacy

We need to keep client_ip and geocoded data for this schema.


This migration will be different than others, as VirtualPageView does not use the EventLogging extension Javascript client to send events. This was done years ago to work around a Do Not Track restriction in EventLogging. This restriction no longer exists, so it should be possible to modify the code in the Popups extension to call mw.eventLog.logEvent instead of sendBeacon directly.

STATUS:
Done.

Related Objects

Event Timeline

Ottomata triaged this task as Medium priority.Nov 14 2019, 6:26 PM
Ottomata moved this task from Incoming to Event Platform on the Analytics board.

Oof. I think VirtualPageView is a bad choice for our first port.
https://phabricator.wikimedia.org/T187277

The Popups extension doesn't use the eventLog.logEvent interface; they built their own and call sendBeacond directly to get around EventLogging's DNT check.

@Ottomata ae you thinking next quarter we will do teh instraumentation of session length but also an additional port of another event?

Not to necessarily port another event, but at least prepare to do so. If things turn out to be easy, then sure, we can port.

We can't do VirtualPageView because they totally bypass EventLogging. Instead, I'll work on T249261: Vertical: Migrate SearchSatisfaction EventLogging event stream to Event Platform.

Sounds good, added the searchsatisfaction ticket to goals.

@sdkim @Ottomata @jlinehan When we migrate this, we need to make sure we include geocoded data. It's important for our high-level metrics (this is included in content interactions, and we look at this by geography).

Also looping in @cchen since she's using this data

@kzimmerman acknowledged. Last speaking to @Ottomata, this would require it's own migration plan and has been slotted at the end of our migration schedule

When we migrate this, we need to make sure we include geocoded data

Great, thanks for the heads up. Can do. :)

Ottomata renamed this task from Vertical: Virtualpageview datastream on MEP to VirtualPageView Event Platform Migration.Jan 28 2021, 4:26 PM
Ottomata reassigned this task from jlinehan to mforns.
Ottomata raised the priority of this task from Medium to High.
Ottomata added a project: Analytics-Kanban.
Ottomata updated the task description. (Show Details)
Ottomata added a subscriber: Jdlrobson.

FYI: the KaiOS-Wikipedia-app is also logging virtual page views so let us know when we should start sending those to event gate.

Oh! Cool. Q: are there any upcoming schema modificiations? If not, we can migrate the schema ASAP, which would enable KaiOS to produce VirtualPageVieww events to EventGate now. If we do that, we'd like to edit protect the metawiki schema, so we don't have to synchronize changes between the systems.

Hi all, FYI, I'd like to migrate this schema in Q4.

Change 685919 had a related patch set uploaded (by Mforns; author: Mforns):

[schemas/event/secondary@master] Add VirtualPageView schema to analytics/legacy

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

Change 685919 merged by jenkins-bot:

[schemas/event/secondary@master] Add VirtualPageView schema to analytics/legacy

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

Change 685928 had a related patch set uploaded (by Mforns; author: Mforns):

[operations/mediawiki-config@master] Migrate VirtualPageView to EventPlatform on testwiki

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

Change 685928 merged by jenkins-bot:

[operations/mediawiki-config@master] Migrate VirtualPageView to EventPlatform on testwiki

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

Mentioned in SAL (#wikimedia-operations) [2021-05-11T18:53:14Z] <otto@deploy1002> Synchronized wmf-config/InitialiseSettings.php: Migrate VirtualPageView to EventPlatform on testwiki - T238138 (duration: 01m 09s)

Change 689205 had a related patch set uploaded (by Mforns; author: Mforns):

[operations/mediawiki-config@master] Migrate VirtualPageView to EventPlatform on all wikis

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

Change 689205 merged by Ottomata:

[operations/mediawiki-config@master] Migrate VirtualPageView to EventPlatform on group 0 and 1 wikis

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

Mentioned in SAL (#wikimedia-operations) [2021-05-17T19:13:34Z] <otto@deploy1002> Synchronized wmf-config/InitialiseSettings.php: Migrate VirtualPageView to Event Platform on group 0 and group 1 - T238138 (duration: 00m 59s)

Change 692611 had a related patch set uploaded (by Mforns; author: Mforns):

[operations/mediawiki-config@master] Migrate VirtualPageView to EventPlatform on all wikis

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

Change 692611 merged by Ottomata:

[operations/mediawiki-config@master] Migrate VirtualPageView to EventPlatform on all wikis

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

Mentioned in SAL (#wikimedia-operations) [2021-05-18T14:38:55Z] <otto@deploy1002> Synchronized wmf-config/InitialiseSettings.php: Migrate VirtualPageView to EventPlatform on all wikis - T238138 (duration: 01m 06s)

Change 693931 had a related patch set uploaded (by Mforns; author: Mforns):

[operations/puppet@production] Refine VirtualPageView schema using eventlogging_legacy job

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

Change 693931 merged by Ottomata:

[operations/puppet@production] Refine VirtualPageView schema using eventlogging_legacy job

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

Change 693934 had a related patch set uploaded (by Mforns; author: Mforns):

[mediawiki/extensions/Popups@master] Set default VirtualPageView version to Event Platform schema URI

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

Change 693934 merged by jenkins-bot:

[mediawiki/extensions/Popups@master] Set default VirtualPageView version to Event Platform schema URI

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

Change 706689 had a related patch set uploaded (by Ottomata; author: Ottomata):

[operations/mediawiki-config@master] Finalize several EventLogging -> Event Platfom migrations

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

Change 706689 merged by jenkins-bot:

[operations/mediawiki-config@master] Finalize several EventLogging -> Event Platfom migrations

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

Mentioned in SAL (#wikimedia-operations) [2021-07-22T19:26:12Z] <otto@deploy1002> Synchronized wmf-config/InitialiseSettings.php: Finalize several EventLogging -> Event Platfom migrations - T282855 T238138 T282562 T271168 (duration: 00m 55s)