Closed Bug 1557706 Opened 5 years ago Closed 5 years ago

return more informative rejection response

Categories

(Socorro :: Antenna, enhancement, P2)

enhancement

Tracking

(Not tracked)

RESOLVED FIXED

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.

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.

Priority: -- → P2
Assignee: nobody → willkg

This is on stage now and it's nice. I'll push it to prod tomorrow.

This is now deployed to prod. Marking as FIXED.

Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED

Making this block bug #1420363 since I'm implementing it to help debug that.

Blocks: 1420363

Oops--I didn't update the systemtests. Reopening this to do that.

Status: RESOLVED → REOPENED
Resolution: FIXED → ---

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.

Status: REOPENED → RESOLVED
Closed: 5 years ago5 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: