Closed Bug 1508335 Opened 3 years ago Closed 3 years ago

drop old db lookup and buildhub lookup in product_versions table in BetaVersionRule

Categories

(Socorro :: Processor, task, P2)

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.
Blocks: 1503383
Depends on: 1507916
Priority: -- → P2
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
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
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
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.