Closed Bug 1377632 Opened 4 years ago Closed 3 years ago

Write the crash-sig in the summary of an auto-generated crash-report in BMO with "@[ ]"

Categories

(Socorro :: Webapp, task, P5)

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: BesTo, Assigned: willkg)

Details

Attachments

(1 file)

Write the crash-signature in the summary-field of an auto-generated crash-report in BMO with "@[ ]".

I think it's better readable and searchable in BMO.

How it e.g. looks now:
Crash in shutdownhang | vcruntime140.dll@0xc574 | huge_ralloc

How it IMHO should looks like:
Crash in [@ shutdownhang | vcruntime140.dll@0xc574 | huge_ralloc ]

Steps to reproduce:
- Open: https://crash-stats.mozilla.com/report/index/9cecca82-95de-4c69-b705-1fcad0170629
- Select under "Bugzilla - Report this bug in" the "Core"-component
I'd like to see some input from Product Integrity before I'd feel comfortable approving this change. It may impact other tooling or convention. 

If that happens and you want to change this, start here:
https://github.com/mozilla-services/socorro/blob/master/webapp-django/crashstats/crashstats/templatetags/jinja_helpers.py#L225
I won't be able to work on this. This bug can be assigned to someone else.
Per Dipankar's request, I'm unassigning this.

Further, this bug was created as a mentored bug, but there's no indication that Adrian offered to mentor it. He's been removing himself as mentor for other bugs, so I'm going to remove him as the mentor for this one, too.

Adrian: If that's not ok, please re-add yourself.

I don't see any urgency here and there are things to work out before any work on this should go forward, so I'm dropping this to a P5.
Assignee: cooldipanks14 → nobody
Mentor: adrian
Status: ASSIGNED → NEW
Priority: -- → P5

Marco, Calixte, Marcia: This bug covers changing the bug summary that gets created when you click on a "Report this bug in" link in the Bugzilla tab of the crash report. Right now it generates summaries like:

Crash in OOM | Small

The reporter is asking to change that to:

Crash in [@ OOM | Small ]

Does that sound ok with the three of you? Do you think changing the summary would break any tooling you know about?

Flags: needinfo?(mozillamarcia.knous)
Flags: needinfo?(mcastelluccio)
Flags: needinfo?(cdenizet)

I can't think of any of our tools that would break with this change.

P.S.: Thinking about it, "Crash in XXX" sounds a bit strange, given that the signature is not always a function.

Flags: needinfo?(mcastelluccio)

I am OK with this change.

Flags: needinfo?(mozillamarcia.knous)

Pushed to prod just now. Marking as FIXED.

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