709166 - Firefox 9.0b5 does not show in advanced query and Admin UI

Status: VERIFIED FIXED

(Socorro :: Webapp, task)

Description

[Marcia Knous [:marcia]]

https://crash-stats.mozilla.com/query?advanced=1

I cannot drill down by platform for 9.0b5 in the advanced query. It does show in the "Current Versions" dropdown - https://crash-stats.mozilla.com/products/Firefox.

Haven't seen this issue before, so filing a bug.

Comment 2

[Robert Kaiser]

This is urgent as we need to look at 9.0b5 data to figure out how the rapidly nearing 9.0 release will fare and if there's any last-minute things we need developers to look at.

Comment 3

[[:jberkus] Josh Berkus]

Verified that 9.0b5 is present in the database and is marked current, and has been present since 12/06.  So this is not a database issue.

Comment 4

[Robert Kaiser]

This now also affects 9.0b6 and we'd need 9.0b5 to be on the front page right now.

Are you sure this version is in all tables the UI is using? Maybe those pages are still using some table that is not considered to be necessary for current use.

Comment 5

[Laura Thomson :laura]

Adrian, can you take a look please?

Comment 6

[[DEACTIVATED] Adrian Gaudebert]

I think this is a caching issue. Brandon recently cached the result of the current products list call from the middleware. I guess the time to live of that cache is not short enough, and should be reduced. 

CCing Brandon, can you please confirm?

Comment 7

[Robert Helmer [:rhelmer]]

Restarting memcached fixes this issue on prod.

It's supposed to refresh every 30 minutes, Brandon is taking a look at this.

Comment 8

[Brandon Savage [:brandon]]

It appears that what the code is doing is pulling the data, using it, and then restoring the same (stale) data with a new TTL. The only way that the data would ever expire is if we had no activity for 30 minutes, which is not the intent.

I have added a flag in https://github.com/mozilla/socorro/pull/221 that triggers a save only when the data is modified in some way. Otherwise the cache will be allowed to expire. The cache is manually expired, when changes are made, as well.

Comment 9

[Marcia Knous [:marcia]]

I see the entries for 9.0.b5 and 9.0b6 in the UI, but when I query for either I get an error.

Comment 10

[[github robot]]

Commit pushed to https://github.com/mozilla/socorro

https://github.com/mozilla/socorro/commit/51771628735a21b8219238825f14cc2ab7401c0a
Merge pull request #221 from brandonsavage/bug709166

Bug 709166 - Adding a flag to check and make sure we are refreshing the data from time to time.

Comment 11

[[github robot]]

Commits pushed to https://github.com/mozilla/socorro

https://github.com/mozilla/socorro/commit/491612a8133722bf687d9d8c7eea5c66f358c3f7
Bug 709166 - Adding a flag to check and make sure we are refreshing the data from time to time.

https://github.com/mozilla/socorro/commit/51771628735a21b8219238825f14cc2ab7401c0a
Merge pull request #221 from brandonsavage/bug709166

Comment 12

[[github robot]]

Commit pushed to https://github.com/mozilla/socorro

https://github.com/mozilla/socorro/commit/d24fa718686e659efed1222966c34965257b66f8
Bug 709166 - Adding a flag to check and make sure we are refreshing the data from time to time.

Comment 13

[Matt Brandt [:mbrandt]]

Verified on stage: I can drill down by platform for 9.0b5 in the advanced query. Results look correct. The expected beta builds are also present in the admin ui.