Closed Bug 1329998 Opened 7 years ago Closed 7 years ago

split save_raw_crash up [antenna]

Categories

(Socorro :: Antenna, task)

task
Not set
normal

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: nobody → willkg
Status: NEW → ASSIGNED
Landed this last week some time. Marking as FIXED.
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
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.

Attachment

General

Created:
Updated:
Size: