Closed Bug 578687 Opened 13 years ago Closed 12 years ago

Add support for Content crashes (tracker)


(Socorro :: General, task)

Not set


(Not tracked)



(Reporter: laura, Assigned: lonnen)



(Whiteboard: Q42011wanted)

Jetpack crashes will be ProcessType=jetpack
Content crashes will be ProcessType=content
Jetpack crashes will also have a JetpackID and a JetpackName

Todo list:
- Add Hbase support (deinspanjer)
- Add PG support (lars or ozten)
- Changes to processor (lars)
- Changes to UI (ozten/ryan/laura)
- Icons for the UI (chowse)

bsmedberg says these will start appearing in about two weeks but we can push this out in 1.8, no need for a special release.

I'll file separate bugs shortly.
Depends on: 580707
Depends on: 580708
Depends on: 580711
Depends on: 580713
Can we attach a raw dump and meta file for the both a Jetpack and a Content crash?
I think the best thing to do for testing now is just take an existing plugin crash report, change the ProcessType=jetpack and ProcessType=content and use that for testing. The final metadata may change a little, but the basics won't. I won't have real product submissions until beta3 at this point, and even that might not have UI.
The meta data is the problem.  If we don't know what it looks like, we can't make appropriate changes to the database to accommodate it.  We're not sure if we need to make an additional table or if it can just be appended to a reports record.  

JetpackID and JetpackName - what are their types and associated data lengths?  Is a JetpackID unique enough to identify a JetpackName?  Or are they unique only when taken together?  

If there are other metadata fields coming too, it might have a major impact on how this gets implemented.  I'm really uncomfortable making database schema changes that could suddenly be found sub-optimal or completely wrong.  It's painful to change the database once it gets pushed to production.
ok, what I want for the short term is no additional metadata other than the existing processtype stuff. I don't think we'll know much about jetpackid/jetpackname until the jetpack guys actually start getting crashes and figure out how they want to correlate and present that data.
(In reply to comment #4)
This sounds like we need to support search by process_type, but we will not support searching for specific Jetpacks by name.

So process_type = 'jetpack' or 'content' in the reports tables and punt on features that would require the metadata.
Target Milestone: 1.8 → 1.9
Target Milestone: 1.9 → 1.8
Target Milestone: 1.8 → 1.9
FYI content process crash reporting has landed for Fennec, bug 581335. They're not currently setting ProcessType=content, I filed bug 614610 on that.
Jetpack crash reports are also going to start being submitted with beta8, at least in some cases.
Can we get some sample crashes, bsmedberg?
What do you mean by sample crashes? Local minidump files, or local JSON files that go with the miniidumps, or crash IDs that have already been submitted?
I think (minidump and json) or (ID for one that has been submitted) would be excellent.


Note that these don't have the ProcessType property in the JSON yet, that still needs to be implemented in bug 614610.
I tried to submit with a correct ProcessType, but it's not responding, so I suspect that socorro isn't accepting ProcessType=jetpack yet.
Depends on: 615327
Target Milestone: 1.9 → 2.2
Depends on: 667030
Depends on: 667033
We need to get as much of this done for 2.1 as possible.
Assignee: nobody → laura
Target Milestone: 2.2 → 2.1
:bsmedberg: I took a look at your e0896b54-26b0-4553-b9c6-bed9c2101129 crash. It did process and correctly shows the process_type == jetpack.  Looking at the raw dump received by the collector however, I don't see any additional meta data in the form of JetpackID and JetpackName.  Is that a breakpad feature that doesn't exist yet?
here's my pending questions that I need with regards to jetpack crashes:

Is a JetpackID unique enough to identify a JetpackName or are they unique only when taken together?  Or is it unique only when the crash ooid/uuid is included?

What are their types and associated data lengths?
Depends on: 671890
Depends on: 671896
Depends on: 671897
Depends on: 671899
I'm not sure what "unique enough to identify a jetpack name" means. Jetpack IDs are the unique identifiers for the extension. The name is just a human-readable extension name so that we don't have to use AMO or something to map IDs back to extensions.

They are both strings of indeterminate length.

But really, we disabled jetpack processes in Firefox 4 beta and haven't turned them back on, so content crashes are by far the higher priority.
you've actually answered the question.  There is a 1:1 relationship between the id and the name.  You could send just the id and we'd still have enough information to associate the crash with a specific jetpack.  Because you're sending both, I was concerned that the relationship wasn't 1:1.

We'll likely implement this with a new 'jetpack' table containing ids and names.  The reports table will have only the id.
It is possible that multiple jetpacks will have the same name (but different IDs), so it's technically 1:many, and its reasonably likely that a jetpack name may change over time. But those are probably not important details...
Depends on: 673546
Jetpack crashes are off the radar for now.

Last remaining Content bugs are slated for 2.2 and later: pushing tracker out.
Summary: Add support for Jetpack and Content crashes (tracker) → Add support for Content crashes (tracker)
Target Milestone: 2.1 → 2.2
Target Milestone: 2.2 → 2.3
Depends on: 688533
Target Milestone: 2.3 → 2.3.1
Target Milestone: 2.3.1 → 2.3.2
Dependent bugs are still open, moving
Whiteboard: Q42011wanted
Assignee: laura → chris.lonnen
Depends on: 697907
Depends on: 698121
Last dependent bug is in a different milestone. Moving.
Target Milestone: 2.3.2 → 2.3.4
Target Milestone: 2.3.4 → 2.4
Depends on: 708868
At long last...
Closed: 12 years ago
Resolution: --- → FIXED
Component: Socorro → General
Product: Webtools → Socorro
Dependent bugs have been verified. Automation and manual bfts are passing.
You need to log in before you can comment on or make changes to this bug.