Closed
Bug 1056025
Opened 10 years ago
Closed 10 years ago
No ADI for FennecAndroid, counted in Fennec instead.
Categories
(Socorro :: Backend, task)
Tracking
(Not tracked)
RESOLVED
FIXED
99
People
(Reporter: kairo, Assigned: rhelmer)
Details
(Whiteboard: [DB Change])
Today we have no FennecAndroid ADI numbers on prod - see e.g. https://crash-stats.mozilla.com/daily?p=FennecAndroid - but we have them in Fennec instead - see https://crash-stats.mozilla.com/daily?p=Fennec
Sounds to me like we are not rewriting ADI data according to the product name + app ID combination like we should.
Reporter | ||
Comment 1•10 years ago
|
||
This is a blocker for Android CrashKill work at this time.
Severity: normal → blocker
Reporter | ||
Comment 2•10 years ago
|
||
WebappRuntime is also missing its ADI and it also ought to be doing a rewrite like that.
Reporter | ||
Comment 3•10 years ago
|
||
Note that we should apply the same rewrite of product name based on the app ID that we do on the crash side also for ADI here, based on the rewrite flag in the product_productid_map table.
Assignee | ||
Updated•10 years ago
|
Status: NEW → ASSIGNED
Comment 4•10 years ago
|
||
Commits pushed to master at https://github.com/mozilla/socorro
https://github.com/mozilla/socorro/commit/6ffa7fcae747e4d67665851abe452cadb4eaf31a
fix bug 1056025 and fix bug 1037855 - rename Fennec->FennecAndroid and Webapp Runtime -> WebappRuntime as appropriate
https://github.com/mozilla/socorro/commit/3a4c2ab324bbd5444f55bdcccef6f31215f59bc0
Merge pull request #2307 from rhelmer/bug1056025-map-fennec-to-fennecandroid
fix bug 1056025 and fix bug 1037855 - rename Fennec->FennecAndroid and W...
Updated•10 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 5•10 years ago
|
||
Migration and backfill run on stage.
Assignee | ||
Updated•10 years ago
|
Whiteboard: [DB Change]
Assignee | ||
Comment 6•10 years ago
|
||
Migration run on prod:
INFO [alembic.migration] Context impl PostgresqlImpl.
INFO [alembic.migration] Will assume transactional DDL.
/data/socorro/socorro-virtualenv/lib/python2.6/site-packages/alembic/util.py:189: UserWarning: Revision 321a815df1c7 referenced from 321a815df1c7 -> 5e14d46c725, bug 855423 change RAISE EXCEPTION to RAISE NOTICE is not present
warnings.warn(msg)
INFO [alembic.migration] Running upgrade 3a36327c2845 -> 52dbc7357409, bug 1056025 and bug 1037855 - rename Fennec->FennecAndroid and WebappRuntime correctly
Updated•10 years ago
|
Target Milestone: --- → 99
You need to log in
before you can comment on or make changes to this bug.
Description
•