Page MenuHomePhabricator

'.search_result_page_id' should be integer
Closed, ResolvedPublicPRODUCTION ERROR

Description

Error
  • mwversion: 1.38.0-wmf.12
normalized_message
'.search_result_page_id' should be integer
Notes

This was originally reported by @elukey on T293953:

Hi everybody,

there is an alert for Eventgate Analytics External event validation errors for the schema mediawiki.mediasearch_interaction, that lines up with the deployment timings:

https://sal.toolforge.org/log/xYr_m30B8Fs0LHO53sGB
Grafana dashboard

Logstash shows the same error over and over: '.search_result_page_id' should be integer

Event Timeline

dancy triaged this task as Unbreak Now! priority.Dec 9 2021, 3:53 PM
dancy updated the task description. (Show Details)

I'm not sure if this should be a train blocker, this is an analytics instrumentation event.

I am not totally sure of the owners of this stream, but @egardner designed the initial schema, so hopefully he can find the culprit. I bet some instrumentation code bug was introduced.

I'm on call for our team, happy to help. This looks like an instrumentation problem, but I'm here in case the obvious fix doesn't work. (ping me on IRC or slack)

I wrote the original analytics code here but am no longer working on the team that maintains MediaSearch. Previously the page ID was used as an identifier whenever the "quickview" element of the UI was activated, and the instrumentation logged the ID to help provide a sense of where & how users were using this feature.

Now the code has been changed and the page title is used instead, so there is no ID to log. This is just a case of the schema getting out of sync with the current UI changes.

It sounds like @Seddon is working on a patch to disable the logging of this event for now (to go out in a backport). But long-term the MediaSearch Interaction schema will need to be updated to reflect how this feature currently works.

wmf.12 is currently already out to test and commons and MediaSearch isn't deployed on any group 2 wiki's so not a train blocker.

Change 745578 had a related patch set uploaded (by Seddon; author: Seddon):

[mediawiki/extensions/MediaSearch@master] Search_result_page_id should be integer

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

Change 745578 merged by jenkins-bot:

[mediawiki/extensions/MediaSearch@master] Search_result_page_id should be integer

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

Change 745646 had a related patch set uploaded (by Clare Ming; author: Seddon):

[mediawiki/extensions/MediaSearch@wmf/1.38.0-wmf.12] Search_result_page_id should be integer

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

Change 745646 merged by jenkins-bot:

[mediawiki/extensions/MediaSearch@wmf/1.38.0-wmf.12] Search_result_page_id should be integer

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

Mentioned in SAL (#wikimedia-operations) [2021-12-10T00:34:35Z] <cjming@deploy1002> Synchronized php-1.38.0-wmf.12/extensions/MediaSearch/resources/components/QuickView.vue: Backport: [[gerrit:745646|Search_result_page_id should be integer (T297400)]] (duration: 00m 55s)

Change 745639 had a related patch set uploaded (by Seddon; author: Seddon):

[mediawiki/extensions/MediaSearch@master] search_result_page_id should be integer

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

wmf.12 is currently already out to test and commons and MediaSearch isn't deployed on any group 2 wiki's so not a train blocker.

@Seddon you sure? Since yesterday the validation error rate jumped a ton and is causing more alerts

Screen Shot 2021-12-10 at 09.27.58.png (1×3 px, 739 KB)

https://grafana.wikimedia.org/goto/OyGWGYh7k

wmf.12 is currently already out to test and commons and MediaSearch isn't deployed on any group 2 wiki's so not a train blocker.

@Seddon you sure? Since yesterday the validation error rate jumped a ton and is causing more alerts

Screen Shot 2021-12-10 at 09.27.58.png (1×3 px, 739 KB)

https://grafana.wikimedia.org/goto/OyGWGYh7k

@Ottomata The second big jump occurred on Wednesday according to that but this was only brought to our attention Thursday so nothing should get worse. The first patch submitted missed something. The second patch if I can't get deployed today will get deployed Monday first thing.

Change 745966 had a related patch set uploaded (by Eric Gardner; author: Eric Gardner):

[schemas/event/secondary@master] Mediasearch_Interaction: Add search_result_page_title field

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

Change 745639 merged by jenkins-bot:

[mediawiki/extensions/MediaSearch@master] Disable event logging for Quickview interactions

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

Change 746810 had a related patch set uploaded (by Urbanecm; author: Seddon):

[mediawiki/extensions/MediaSearch@wmf/1.37.0-wmf.12] Disable event logging for Quickview interactions

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

Change 746810 abandoned by Urbanecm:

[mediawiki/extensions/MediaSearch@wmf/1.37.0-wmf.12] Disable event logging for Quickview interactions

Reason:

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

Change 746811 had a related patch set uploaded (by Urbanecm; author: Seddon):

[mediawiki/extensions/MediaSearch@wmf/1.38.0-wmf.12] Disable event logging for Quickview interactions

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

Change 746811 merged by jenkins-bot:

[mediawiki/extensions/MediaSearch@wmf/1.38.0-wmf.12] Disable event logging for Quickview interactions

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

Mentioned in SAL (#wikimedia-operations) [2021-12-13T12:25:28Z] <urbanecm@deploy1002> Synchronized php-1.38.0-wmf.12/extensions/MediaSearch/resources/components/SearchResults.vue: 7d0fa97dcfe4f3c0b5a5f98e722f54930c18bad0: Disable event logging for Quickview interactions (T297400) (duration: 00m 56s)

Update from @Seddon:

We disabled the logging error back in mid-December before I went on leave. Eric has submitted a patch for a schema update which needs code reviewing. Unfortunately we don't though have event logging as standard as part of our dev environment it seems and getting it fully working locally has proven troublesome. Matthias is helping with that and I've also pinged Eric to try and get to the bottom of the issue. Once that is resolved I can complete the CR and push out the schema change. After the schema change is done we can reenable the event that we suppressed.

Change 745966 merged by jenkins-bot:

[schemas/event/secondary@master] Mediasearch_Interaction: Add search_result_page_title field

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

Schema update has been merged. Will write a patch to update the logging in MediaSearch.

Change 754611 had a related patch set uploaded (by Seddon; author: Seddon):

[mediawiki/extensions/MediaSearch@master] Update logging. New schema. Page id to title

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

Change 754611 merged by jenkins-bot:

[mediawiki/extensions/MediaSearch@master] Update logging. New schema. Page id to title

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

Seddon lowered the priority of this task from Unbreak Now! to High.Jan 25 2022, 4:21 AM