Many logged in users run custom gadgets. They contribute a lot of noise to the error dashboards in Logstash.
To make this data more usable and put less burden on developers to filter out this noise and find bugs that impact the majority of our anonymous users, I request that a field isloggedin in the response to filter out errors from logged in users.
Change 632741 had a related patch set uploaded (by Jason Linehan; owner: Jason Linehan):
[mediawiki/extensions/WikimediaEvents@master] clientError: Adds 'is_logged_in' tag to aid filtering
Change 632741 merged by jenkins-bot:
[mediawiki/extensions/WikimediaEvents@master] clientError: Adds 'is_logged_in' tag to aid filtering
I believed this code should be live now (wmf13 is deployed on commons wiki) but I'm not seeing any errors with the tag. Is a further change needed?
We seem to have lost client side error tracking because of this change, so we should fix that before rolling out wmf13 any further so we don't lose it altogether.
Change 634120 had a related patch set uploaded (by Jason Linehan; owner: Jason Linehan):
[mediawiki/extensions/WikimediaEvents@master] clientError: Convert bool to string to allow schema validation
Change 634124 had a related patch set uploaded (by Jason Linehan; owner: Jason Linehan):
[schemas/event/primary@master] mediawiki/client/error: Adds is_logged_in field
I believe this issue is due to the event being invalid due to the fact that the values of members of the tags object must be strings per the schema, but we were providing a boolean value for is_logged_in. Being mindful of https://phabricator.wikimedia.org/T217142#5295424, plus the fact that there is already a 'tags' field being used in logstash somehow, I've updated the schema to have a top-level is_logged_in field, which takes a boolean value.
Change 634120 merged by jenkins-bot:
[mediawiki/extensions/WikimediaEvents@master] clientError: Add is_logged_in field rather than tags field
Thanks for addressing the logspam; much appreciated! However, is this an actual train blocker? I.e. should I be waiting on a backport to wmf/1.36.0-wmf.13 before promoting to all wikis today? If not, please remove T263179: 1.36.0-wmf.13 deployment blockers as a parent task.
Change 634256 had a related patch set uploaded (by Jdlrobson; owner: Jdlrobson):
[mediawiki/extensions/WikimediaEvents@wmf/1.36.0-wmf.13] Revert "clientError: Adds 'is_logged_in' tag to aid filtering"
I think we should be waiting on a backport. If the train rolls out further we'll lose a key bit of infrastructure (client-side error logging). This deploy is getting a little complex and we need more time.
As a short term solution and to unblock the wmf13 deploy, let's revert it in wmf13 and then move this to a wmf14 blocker
Change 634292 had a related patch set uploaded (by Jason Linehan; owner: Jason Linehan):
[schemas/event/primary@master] mediawiki/client/error: Adds is_user_logged_in field
Change 634124 abandoned by Jason Linehan:
[schemas/event/primary@master] mediawiki/client/error: Adds is_logged_in field
Reason:
Change 634295 had a related patch set uploaded (by Jason Linehan; owner: Jason Linehan):
[mediawiki/extensions/WikimediaEvents@master] clientError: Adds 'is_user_logged_in' field.
Thanks for clarifying that. I'll make sure the backport is merged/synced before going to all wikis.
Change 634256 merged by jenkins-bot:
[mediawiki/extensions/WikimediaEvents@wmf/1.36.0-wmf.13] Revert "clientError: Adds 'is_logged_in' tag to aid filtering"
Mentioned in SAL (#wikimedia-operations) [2020-10-15T19:07:16Z] <catrope@deploy1001> Synchronized php-1.36.0-wmf.13/extensions/WikimediaEvents/: Revert "clientError: Adds is_logged_in tag to aid filtering" (T256173) (duration: 01m 58s)
https://gerrit.wikimedia.org/r/c/mediawiki/extensions/WikimediaEvents/+/635082 will unblock 1.36.0-wmf.14
Change 639529 had a related patch set uploaded (by Jason Linehan; owner: Jason Linehan):
[schemas/event/primary@master] mediawiki/client/error: Change fields to help with Logstash.
Change 634292 abandoned by Jason Linehan:
[schemas/event/primary@master] mediawiki/client/error: Adds is_user_logged_in and is_banner_shown.
Reason:
Superceded by https://gerrit.wikimedia.org/r/c/schemas/event/primary/ /639529
Change 639529 abandoned by Jason Linehan:
[schemas/event/primary@master] mediawiki/client/error: Change fields to help with Logstash.
Reason:
Abandoned since we're trying to be more ECS-like for these. See: https://gerrit.wikimedia.org/r/c/schemas/event/primary/ /647025
Change 634295 abandoned by Jdlrobson:
[mediawiki/extensions/WikimediaEvents@master] clientError: Adds new fields and updates behavior.
Reason:
No longer being worked on
Change 829299 had a related patch set uploaded (by Gergő Tisza; author: Gergő Tisza):
[mediawiki/extensions/WikimediaEvents@master] clientError: Add more information to log events
Change 829299 merged by jenkins-bot:
[mediawiki/extensions/WikimediaEvents@master] clientError: Add more information to log events