667030 - Possibility to search for content crashes

Status: VERIFIED FIXED

(Socorro :: General, task)

Description

[Robert Kaiser]

Fennec already lists content crashes, and Firefox will get those as well some time, hopefully later this year. (Bug 578687 also talks about jetpack processes, but I have no clue on the plans there right now, let's keep in mind that those might come as well and build stuff in a way so those can be added easily.)

A first step to improving support for those would be to be able to search for process_type=content from advanced search.

Comment 1

[Laura Thomson :laura]

This should go into 2.3.3, but we don't have a 2.3.3 yet.

Comment 2

[Laura Thomson :laura]

By which I mean "retriage at that stage" - may be done with ES, maybe with reports_clean, or maybe with newtcbs.

Comment 3

[Lonnen :lonnen]

Spun off the elastic search as an implementation in a separate bug https://bugzilla.mozilla.org/show_bug.cgi?id=698121, which is blocking the tracker for content crashes but shouldn't block this.

pull: https://github.com/mozilla/socorro/pull/117
adrian: r?

Comment 4

[[DEACTIVATED] Adrian Gaudebert]

The number problem that I saw yesterday was explained to me by jberkus: it's due to search counting unique signatures, so if 2 crashes with the same signature have different process types, that signature will appear in both types but will be counted only once in "any". 

r+ and merged.

Comment 5

[[github robot]]

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

https://github.com/mozilla/socorro/commit/4efd1c7fc9c50a0de9e9e5b7eb1f1949434f2a4e
Merge pull request #117 from Lonnen/667030-search

add content as a recognized option in the postgres implementation, add a 
Fixes bug 667030

Comment 6

[[DEACTIVATED] Adrian Gaudebert]

I am very sorry I merged something that is not fully working. I forgot to test the topcrashers part as there was a change there too. When I hit the "content" type link I get that error: http://pastebin.mozilla.org/1366809

Comment 7

[Lonnen :lonnen]

The config change was already committed here: https://github.com/mozilla/socorro/pull/116

It was accidentally also included in the pull request I asked you to review.

The error occurs because the middleware relies on an additional function that isn't in your dump of the db yet. You can put it there by running the contents of (https://github.com/mozilla/socorro/blob/b954d625fd54c168643d433ceb5ad155e0cbd36c/sql/upgrade/2.3.3/update_tcbs.sql) in your psql shell. After that, it should work fine.

Comment 8

[Lonnen :lonnen]

Since this is working on dev (https://crash-stats-dev.allizom.org/topcrasher/byversion/Firefox/10.0a1/7/content) I'm going to re-mark it as fixed. Adrian, please re-open if you run the migration script and get the same error.

Comment 9

[Matt Brandt [:mbrandt]]

QA verified. Content crashes are viewable.