Closed
Bug 1329998
Opened 7 years ago
Closed 7 years ago
split save_raw_crash up [antenna]
Categories
(Socorro :: Antenna, task)
Socorro
Antenna
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: willkg, Assigned: willkg)
References
Details
Attachments
(1 file)
Antenna's save_raw_crash saves raw crash metadata but also a bunch of dumps to the crash storage's store. I originally thought that all these things were part of one single unit and that it made sense to save the unit all at once every time. But maybe that's not the case. We have the idea of "this is the crash report we originally got from the client" which has a "raw_crash" part and a bunch of dumps. Later in processing, we have a "processed crash" which is the result of processing a raw crash, though in actuality, we have a bunch of different possible outputs from processing a raw crash. Those are almost entirely metadata related and don't touch the dumps. Maybe it makes sense to think of a crash report as "the original data we got from the client" plus a bunch of dumps which don't usually get updated. So then it makes sense to split save_raw_crash into save_raw_crash and maybe save_dump and have the caller iterate over all the dumps to be saved and handle the (None, '', 'upload_file_minidump') -> 'dump' name change, too. This bug covers mulling over that some more and then making whatever changes seem appropriate.
Assignee | ||
Comment 1•7 years ago
|
||
Assignee | ||
Updated•7 years ago
|
Assignee: nobody → willkg
Status: NEW → ASSIGNED
Assignee | ||
Comment 2•7 years ago
|
||
Landed this last week some time. Marking as FIXED.
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 3•7 years ago
|
||
Switching Antenna bugs to Antenna component.
Component: General → Antenna
You need to log in
before you can comment on or make changes to this bug.
Description
•