Find a way to make /daily work with new "Android" reporting in the short term

VERIFIED FIXED in 22

Status

Socorro
General
--
critical
VERIFIED FIXED
5 years ago
5 years ago

People

(Reporter: Robert Kaiser, Assigned: selenamarie)

Tracking

unspecified
ARM
Android

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [qa-])

(Reporter)

Description

5 years ago
If you look at  https://crash-stats.mozilla.com/daily?form_selection=by_version&p=FennecAndroid&v[]=18.0a1&v[]=17.0a2&v[]=16.0b4&v[]=15.0.1&hang_type=any&os[]=Windows&os[]=Mac+OS+X&os[]=Linux&date_range_type=build&date_start=2012-09-13&date_end=2012-09-27&submit=Generate you'll notice that starting with 2012-09-26 this reports you no users.

This is because the new Breakpad version we have in there now reports the OS to be "Android" instead of Linux, see a sample crash report at bp-d0213091-5c3d-42ef-a747-04f942120927 and so they don't get counted against "Linux" there any more. Note that ADUs still get reported as "Linux" though,so this doesn't match.

We should do a short-term fix here to have them somehow show up as "Linux" in the UI (I guess the fix might actually be MWare or DB but I'll leave that up to you guys), until we have the possibility to show Android as Android in future UI.
Selena,

Adding a row to os_name_matches would map Android to Linux.

Updated

5 years ago
Assignee: nobody → sdeckelmann
(Reporter)

Comment 2

5 years ago
Can we please get some progress here?
This is significantly impairing our ability to assess Android stability, and the change that caused this will also make its way into Aurora early next week.

Updated

5 years ago
Target Milestone: --- → 22

Updated

5 years ago
Severity: normal → critical
Did this on .stage: 

 insert into os_name_matches values('Linux', 'Android%'); 

Will try to verify that this had the intended effect on /daily per KaiRo's IRC comments.
Depends on: 797566
(In reply to Selena Deckelmann :selena from comment #3)
> Did this on .stage: 
Just checking in -- is the bug ready to be bumped to fixed?
(Reporter)

Comment 5

5 years ago
(In reply to Selena Deckelmann :selena from comment #3)
> Did this on .stage: 
> 
>  insert into os_name_matches values('Linux', 'Android%'); 
> 
> Will try to verify that this had the intended effect on /daily per KaiRo's
> IRC comments.

Seems not to be the case. :(

This is especially visible on https://crash-stats.allizom.org/daily?form_selection=by_version&p=FennecAndroid&v[]=18.0a1&v[]=17.0a2&v[]=16.0b6&v[]=15.0.1&hang_type=any&os[]=Windows&os[]=Mac+OS+X&os[]=Linux&date_range_type=build&date_start=2012-09-23&date_end=2012-10-07&submit=Generate - we should have numbers for crashes on 18.0a1 for all days, but due to this bug, it goes to 0 starting with 2012-09-26.
This may just be because stage got rebuilt without the data change. 

I'm re-adding it and re-running backfill_matviews to see if this works now.
https://github.com/mozilla/socorro/pull/872 -- solves problems going forward, but to fix old reports, we need to do some backfilling
Status: NEW → ASSIGNED
Whiteboard: [qa-]
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
To prove this worked with a recent matview run: 

select uuid, os_name from reports_clean where uuid in ( select uuid from reports where date_processed > '2012-10-10' and os_name = 'Android' limit 1);

os_name should be "Linux"

Comment 9

5 years ago
Commits pushed to master at https://github.com/mozilla/socorro

https://github.com/mozilla/socorro/commit/0af9875cdddc19d88ad6f648f4997d05a87f2ca2
bug 795349 fix for Fennec

https://github.com/mozilla/socorro/commit/837848d73d6ff4ae5176396c43eade0f419cc23f
Merge pull request #872 from selenamarie/795349-tiny-db-fix

bug 795349 fix for Fennec
(Reporter)

Comment 10

5 years ago
Verified by the data for yesterday having the Android data in the Linux columns.
Status: RESOLVED → VERIFIED
Great success -- thank you kairo.

Comment 12

5 years ago
Graphs per builddate for Android crash signatures still show a null line. See https://crash-stats.mozilla.com/report/list?signature=java.lang.NullPointerException%3A%20at%20org.mozilla.gecko.Tabs.loadUrlInTab%28Tabs.java%29#graph
You need to log in before you can comment on or make changes to this bug.