return more informative rejection response
Categories
(Socorro :: Antenna, enhancement, P2)
Tracking
(Not tracked)
People
(Reporter: willkg, Assigned: willkg)
References
Details
Attachments
(3 files)
Currently, Antenna returns Discarded=1 if the crash report is rejected. That's great in most cases except when you're working on a crash reporter issue and trying to figure out why crash reports are getting rejected.
Maybe change it to Discarded=REASON. We can re-use the metrics keys. We'd want to have a table somewhere explaining the reasons.
This bug covers changing the rejection response to something more helpful.
|
Assignee | |
Comment 1•5 years ago
|
||
This is currently blocking Gabriele, so I'm making it a P2 to do soon. I think it's straight-forward, but it might not be depending on whether Antenna knows the reason for all cases at the rejection response return.
|
|
Assignee | |
Comment 2•5 years ago
|
||
|
|
Assignee | |
Comment 3•5 years ago
|
||
|
|
Assignee | |
Comment 4•5 years ago
|
||
|
|
Assignee | |
Comment 5•5 years ago
|
||
This is on stage now and it's nice. I'll push it to prod tomorrow.
|
|
Assignee | |
Comment 6•5 years ago
|
||
This is now deployed to prod. Marking as FIXED.
|
|
Assignee | |
Comment 7•5 years ago
|
||
Making this block bug #1420363 since I'm implementing it to help debug that.
|
|
Assignee | |
Comment 8•5 years ago
|
||
Oops--I didn't update the systemtests. Reopening this to do that.
|
|
Assignee | |
Comment 9•5 years ago
|
||
|
|
Assignee | |
Comment 10•5 years ago
|
||
willkg merged PR #354: "bug 1557706: fix systemtests to work with discarded reasons" in aa793a0.
This only affects system tests (minus a tweak to malformed crash handling), so I'm going to mark it FIXED now.
Description
•