Closed Bug 951825 Opened 11 years ago Closed 11 years ago

Exploitability tables in Signature Summary are wrong/broken

Categories

(Socorro :: Webapp, task)

task
Not set
normal

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: kairo, Assigned: selenamarie)

References

Details

(Whiteboard: [DB Changes][qa-])

Attachments

(1 file)

I'm adding Selena, Peter, and Brandon here because they have done some work on exploitability recently and something probably regressed this.

When looking at the Exploitability section of Signature summary, 1) I see the same values for all of null/low/medium/high ratings on all rows and 2) I see multiple rows per day - both of which is wrong/broken.

One example I am looking at is https://crash-stats.mozilla.com/report/list?signature=CryptBaseRunOnceRoutine
Commits pushed to master at https://github.com/mozilla/socorro

https://github.com/mozilla/socorro/commit/52bcdb29171d06459fc7ef5402a3215650bb001c
Fixes Bug 951825 - Resolves issue where reports aren't being grouped by the report date.

https://github.com/mozilla/socorro/commit/d9199b7c23814b1862bd46cacad97d37f7babfd8
Merge pull request #1753 from brandonsavage/951825_exploitability_broken

Fixes Bug 951825 - Resolves issue where reports aren't being grouped by ...
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Assignee: nobody → bsavage
Target Milestone: --- → 70
Still broken.
Assignee: bsavage → peterbe
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
The problem is not with the middleware. The underlying data in the matview is incorrect. I have a patch, I think.
Commits pushed to master at https://github.com/mozilla/socorro

https://github.com/mozilla/socorro/commit/9282a3761dd191770028e0627c7ca65dbd5aa858
Bug 951825 - Use sum() instead of count() to figure exploitability counts

https://github.com/mozilla/socorro/commit/45fe586ab62fe8b93c134dbc6448791056ff4e7f
Merge pull request #1767 from bsmedberg/bug951825

Bug 951825 - Use sum() instead of count() to figure exploitability counts
Needs a migration.
Target Milestone: 70 → 72
Attachment #8356587 - Flags: review?(benjamin)
Commits pushed to master at https://github.com/mozilla/socorro

https://github.com/mozilla/socorro/commit/f9c50959ce0f34222a5ae46fdb52b607cf0999c3
Fixes bug 951825 migration for update_exploitability.sql

https://github.com/mozilla/socorro/commit/bc4dffb162e5882065c55a7638d695fff6a29340
Merge pull request #1768 from selenamarie/bug951825-migration-for-expoitability-matview

Fixes bug 951825 migration for update_exploitability.sql
Status: REOPENED → RESOLVED
Closed: 11 years ago11 years ago
Resolution: --- → FIXED
Awesome. When does this change hit production?
We push releases most Wednesdays. Probably tomorrow, certainly within 8 days.
Target Milestone: 72 → 71
Whiteboard: [DB Changes][qa-]
Assignee: peterbe → sdeckelmann
Bumping to verified on stage as [qa-]. This hasn't landed on production yet but will soon.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: