Closed
Bug 1122145
Opened 9 years ago
Closed 9 years ago
Socorro not picking up Fennec 38.0a1 and 37.0a2
Categories
(Socorro :: Database, task)
Socorro
Database
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: rhelmer, Assigned: selenamarie)
References
Details
Fennec 37.0a2 and 38.0a1 are posted to FTP but Socorro hasn't picked them up yet.
Reporter | ||
Comment 1•9 years ago
|
||
I think the problem is that update_product_versions() does a join on release_repositories, and looking in releases_raw we're seeing some new entries in the repository column: mozilla-central-android-x86 mozilla-central-android-api-9 mozilla-central-android-api-11 mozilla-aurora-android-x86 mozilla-aurora-android-api-9 mozilla-aurora-android-api-11
Reporter | ||
Comment 2•9 years ago
|
||
(In reply to Robert Helmer [:rhelmer] from comment #1) > I think the problem is that update_product_versions() does a join on > release_repositories, and looking in releases_raw we're seeing some new > entries in the repository column: > > mozilla-central-android-x86 > mozilla-central-android-api-9 > mozilla-central-android-api-11 > mozilla-aurora-android-x86 > mozilla-aurora-android-api-9 > mozilla-aurora-android-api-11 Kairo - there are multiple builds for nightly and aurora showing up, and Socorro doesn't recognize any of these repositories (which is how it determines which of these buildids should be treated as 38.0a1 in the UI for instance) Any idea which of these repos we should pick?
Flags: needinfo?(kairo)
Reporter | ||
Updated•9 years ago
|
Assignee: nobody → rhelmer
Status: NEW → ASSIGNED
Comment 3•9 years ago
|
||
I know we split Android builds into multiple for different OS versions, that's probably behind this. Mark, can you help with Rob's questions?
Flags: needinfo?(kairo) → needinfo?(mark.finkle)
Comment 4•9 years ago
|
||
I'm not exactly sure what Rob's question is, but yes, we split the APKs into android-api-9 and android-api-11 APKs, starting in Fx37. There is no longer a single adroid ARMv7 APK. What other details can we provide to help out? CC'ing Richard and Justin to be around for any followup questions.
Flags: needinfo?(mark.finkle)
Comment 5•9 years ago
|
||
The question is which of those APKs should determine which build IDs and versions we should look up on FTP to be able to determine which FennecAndroid versions exist.
Comment 6•9 years ago
|
||
I believe the build IDs and versions should be the same; these will be two jobs triggered from the same revision at the same time. ftp://ftp.mozilla.org/pub/mobile/nightly/latest-mozilla-aurora-android-api-11/fennec-37.0a2.multi.android-arm.json and ftp://ftp.mozilla.org/pub/mobile/nightly/latest-mozilla-aurora-android-api-9/fennec-37.0a2.multi.android-arm.json have the same build ID. (The real answer is "every APK", but you can settle for just looking at one.)
Comment 7•9 years ago
|
||
If we go for one, I think going for the api-11 is probably better as it's the newer platform.
Reporter | ||
Comment 8•9 years ago
|
||
OK thanks, this answers my question.
Comment 9•9 years ago
|
||
Commits pushed to master at https://github.com/mozilla/socorro https://github.com/mozilla/socorro/commit/362a7fb9ac3c790788d48cd48b148edca15b0ae4 Fixes bug 1122145 - add fennec repos to table https://github.com/mozilla/socorro/commit/f64b3677d4649deb3db39b80c1e672c14d48a885 Merge pull request #2580 from selenamarie/bug1122145-add-release-repos-for-fennec Fixes bug 1122145 - add fennec repos to table
Updated•9 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Reporter | ||
Updated•9 years ago
|
Assignee: rhelmer → sdeckelmann
Reporter | ||
Comment 10•9 years ago
|
||
The fix for this will be pushed to production shortly.
Reporter | ||
Comment 11•9 years ago
|
||
This is in production and these new releases should show up when reports are generated early tomorrow morning.
Comment 12•9 years ago
|
||
Still not seeing Fennec 38 on crash-stats (at least on this canonical URL https://crash-stats.mozilla.com/topcrasher/products/FennecAndroid/versions/38.0a1). Can we double check here?
Comment 13•9 years ago
|
||
(In reply to Aaron Train [:aaronmt] from comment #12) > Still not seeing Fennec 38 on crash-stats (at least on this canonical URL > https://crash-stats.mozilla.com/topcrasher/products/FennecAndroid/versions/ > 38.0a1). Can we double check here? Ditto 37.0a2
Reporter | ||
Comment 14•9 years ago
|
||
FennecAndroid 37.0a2 and 38.0a1 still aren't showing up in product_versions, I'll take a look too
Status: RESOLVED → REOPENED
Flags: needinfo?(sdeckelmann)
Resolution: FIXED → ---
Reporter | ||
Comment 15•9 years ago
|
||
There's also a special_product_platforms table, since this product has a different name in the ftp metadata ("mobile") and in the UI ("FennecAndroid"). It also has a "repository" column that needs to be updated.
Comment 16•9 years ago
|
||
Commits pushed to master at https://github.com/mozilla/socorro https://github.com/mozilla/socorro/commit/6fae9ed2964f114746aa287bc9fd1ee9e33e053b Fixes bug 1122145 add fennec release repos to special_product_platforms https://github.com/mozilla/socorro/commit/5294ec2c1e57cd8238e789af6dfbe7bd5e62f7b0 Merge pull request #2587 from selenamarie/bug1122145-fennec-join-table-update Fixes bug 1122145 add fennec release repos to special_product_platforms
Updated•9 years ago
|
Status: REOPENED → RESOLVED
Closed: 9 years ago → 9 years ago
Resolution: --- → FIXED
Comment 17•9 years ago
|
||
When should we start to see data for Fennec 38.0a1 and Fennec 37.0a2 show up on the dashboard?
Reporter | ||
Comment 18•9 years ago
|
||
(In reply to Mark Finkle (:mfinkle) from comment #17) > When should we start to see data for Fennec 38.0a1 and Fennec 37.0a2 show up > on the dashboard? This fix is going out this morning, I should be able to backfill so it'll show up today.
Reporter | ||
Updated•9 years ago
|
Flags: needinfo?(sdeckelmann)
Reporter | ||
Comment 19•9 years ago
|
||
Fix is pushed to production, and the new versions are showing up now. I am backfilling reports for the 13th to the 20th so there should be data there too in the next few hours, will update this bug when it's ready.
Reporter | ||
Comment 20•9 years ago
|
||
(In reply to Robert Helmer [:rhelmer] from comment #19) > Fix is pushed to production, and the new versions are showing up now. > > I am backfilling reports for the 13th to the 20th so there should be data > there too in the next few hours, will update this bug when it's ready. Took a bit longer than expected, but we have data for the last week now too: https://crash-stats.mozilla.com/home/products/FennecAndroid
You need to log in
before you can comment on or make changes to this bug.
Description
•