Closed
Bug 1508335
Opened 5 years ago
Closed 5 years ago
drop old db lookup and buildhub lookup in product_versions table in BetaVersionRule
Categories
(Socorro :: Processor, task, P2)
Socorro
Processor
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: willkg, Assigned: willkg)
References
Details
Attachments
(2 files)
We're replacing the product_versions and product_version_builds tables with a crashstats_productversion table that has the information we need to do the BetaVersionRule lookup, is managed by Django, and is fed by the archivescraper crontabber app. Once we finish fixing bug #1507916, archivescraper and crashstats_productversion should do fine and we can switch the BetaVersionRule to use the crashstats_productversion table as a backup to Buildhub and stop using product_versions. This bug covers that work.
Assignee | ||
Updated•5 years ago
|
Assignee | ||
Updated•5 years ago
|
Priority: -- → P2
Assignee | ||
Comment 1•5 years ago
|
||
Buildhub is missing data for Firefox versions 40 through 49. Additionally, it has architecture problems which are the reason for a complete rewrite as Buildhub2. The data we have in the crashstats_productversion table can handle all the BetaVersionRule lookups. We can fix it easily if there are issues. Given that we have to maintain our own crashstats_productversion table anyhow and Buildhub is being rewritten, it makes sense to drop our Buildhub code and re-implement it later when Buildhub2 is ready to go. That gets Socorro to a stable situation now with less technical debt. I'm going to have this bug cover removing the Buildhub bits.
Assignee: nobody → willkg
Status: NEW → ASSIGNED
Summary: drop lookup in product_versions table in BetaVersionRule → drop old db lookup and buildhub lookup in product_versions table in BetaVersionRule
Assignee | ||
Comment 2•5 years ago
|
||
NB: The Buildhub issue is this one: https://github.com/mozilla-services/buildhub/issues/528
Assignee | ||
Comment 3•5 years ago
|
||
Comment 4•5 years ago
|
||
Commits pushed to master at https://github.com/mozilla-services/socorro https://github.com/mozilla-services/socorro/commit/40e256498d0744a6467b2c6043e24fffc6e79ed5 fix bug 1508335: remove buildhub/old-db lookup from BetaVersionRule This reworks the BetaVersionRule to rely solely on looking up information in the crashstats_productversion table and not Buildhub or product_versions. https://github.com/mozilla-services/socorro/commit/6cd879b719476dad795b2e882f32aebbed8a8723 Merge pull request #4721 from willkg/1508335-bvr fix bug 1508335: clean up BetaVersionRule lookups
Updated•5 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 5•5 years ago
|
||
Comment 6•5 years ago
|
||
Commits pushed to master at https://github.com/mozilla-services/socorro https://github.com/mozilla-services/socorro/commit/8a3a477f0067620c5d0b47aa92d3362c9433ffe4 bug 1508335: add logging for failed lookups in BetaVersionRule https://github.com/mozilla-services/socorro/commit/c8c16fe681a698dcea690f0af23897631b745e61 Merge pull request #4722 from willkg/1508335-bvr-logging bug 1508335: add logging for failed lookups in BetaVersionRule
You need to log in
before you can comment on or make changes to this bug.
Description
•