Page MenuHomePhabricator
Create Task
Maniphest T121772

isTestA should always be logged with MobileWebSectionUsage schema
Closed, ResolvedPublic
Actions

Description

Without this we are not going to get all the data we hoped for. Is someone able to look into this in the hope of swatting tonight?

Details

SubjectRepoBranchLines +/-
Schema:MobileWebSectionUsage: always log the isTestA fieldmediawiki/extensions/MobileFrontendwmf/1.27.0-wmf.9+19 -17
Schema:MobileWebSectionUsage: always log the isTestA fieldmediawiki/extensions/MobileFrontendmaster+19 -17
Customize query in gerrit

Related Objects
Search...

Event Timeline

Jdlrobson created this task.Dec 17 2015, 4:41 PM
Jdlrobson assigned this task to • bmansurov.
Jdlrobson raised the priority of this task from to Unbreak Now!.
Jdlrobson updated the task description. (Show Details)
Jdlrobson added projects: MobileFrontend, Reading Web Sprint 62 - DJ-Jazzy-Jeff-and-the-Fresh-Sprints.
Jdlrobson added subscribers: phuedx, gerritbot, • Tbayer and 2 others.
• bmansurov renamed this task from isTestA should always be logged with MobileSectionUsage schema to isTestA should always be logged with MobileWebSectionUsage schema.Dec 17 2015, 4:50 PM
• bmansurov moved this task from Needs Analysis to Doing on the Reading Web Sprint 62 - DJ-Jazzy-Jeff-and-the-Fresh-Sprints board.
• bmansurov set Security to None.
gerritbot added a comment.Dec 17 2015, 5:25 PM

Change 259755 had a related patch set uploaded (by Bmansurov):
Schema:MobileWebSectionUsage: always log the isTestA field

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

gerritbot added a project: Patch-For-Review.Dec 17 2015, 5:25 PM
• bmansurov moved this task from Doing to Code Review on the Reading Web Sprint 62 - DJ-Jazzy-Jeff-and-the-Fresh-Sprints board.Dec 17 2015, 5:25 PM
gerritbot added a comment.Dec 17 2015, 5:40 PM

Change 259761 had a related patch set uploaded (by Jdlrobson):
Schema:MobileWebSectionUsage: always log the isTestA field

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

gerritbot added a comment.Dec 17 2015, 5:44 PM

Change 259755 merged by jenkins-bot:
Schema:MobileWebSectionUsage: always log the isTestA field

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

phuedx moved this task from Code Review to Ready for Signoff on the Reading Web Sprint 62 - DJ-Jazzy-Jeff-and-the-Fresh-Sprints board.Dec 17 2015, 5:46 PM
ReleaseTaggerBot added a project: MW-1.27-release (WMF-deploy-2016-01-12_(1.27.0-wmf.10)).Dec 17 2015, 6:00 PM
Jdlrobson removed a project: Patch-For-Review.Dec 17 2015, 6:21 PM
gerritbot added a comment.Dec 18 2015, 12:07 AM

Change 259761 merged by jenkins-bot:
Schema:MobileWebSectionUsage: always log the isTestA field

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

Jdlrobson closed this task as Resolved.Dec 18 2015, 12:26 AM

I manually forced myself to always have events logged:
https://en.m.wikipedia.org/wiki/Special:MobileDiff/695708002

I can thus confirm that the isTestA is being set for other events but haven't been able to confirm in EL tables.

select count(*) from MobileWebSectionUsage_15038458 where event_eventName= 'entered' and event_isTestA is not null
0

Will check again tomorrow when the database has caught up before signing off.

Thanks all for the fast response!

ReleaseTaggerBot added a project: MW-1.27-release (WMF-deploy-2015-12-15_(1.27.0-wmf.9)).Dec 18 2015, 1:01 AM
Jdlrobson moved this task from Ready for Signoff to Done on the Reading Web Sprint 62 - DJ-Jazzy-Jeff-and-the-Fresh-Sprints board.Dec 18 2015, 1:09 AM
Content licensed under Creative Commons Attribution-ShareAlike (CC BY-SA) 4.0 unless otherwise noted; code licensed under GNU General Public License (GPL) 2.0 or later and other open source licenses. By using this site, you agree to the Terms of Use, Privacy Policy, and Code of Conduct. · Wikimedia Foundation · Privacy Policy · Code of Conduct · Terms of Use · Disclaimer · CC-BY-SA · GPL · Credits