Closed Bug 1762207 Opened 11 months ago Closed 10 months ago

remove hang_type

Categories

(Socorro :: Processor, task, P3)

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: willkg, Assigned: willkg)

Details

Attachments

(1 file)

I was looking at the PluginRule and noticed the hang_type code makes no sense.

I investigated further and every single crash report indexed in Elasticsearch has a hang_type of 0.

hang_type is determined from the Hang and HangID annotations in this code:

https://github.com/mozilla-services/socorro/blob/20028cd706bb1308eafc6f3e34fdff1a2db1790a/socorro/processor/rules/mozilla.py#L263-L279

CrashAnnotations.yaml doesn't list either annotation. My skimming of mozilla-central suggests that it's not an omission in CrashAnnotations.yaml, but that neither are emitted in crash reports.

This bug covers verifying that the annotations aren't emitted and hang_type is not a thing and removing them.

Gabriele: Are Hang and HangID annotations emitted anymore? If not, I'll just remove this code.

Flags: needinfo?(gsvelto)

That's a very good question. I don't remember where those lived, I suppose it was in the Flash plugin code but I haven't seen them around in a while so I suspect they've been removed a long time ago. I'm going to dig in the history to see where they were gone.

From what I can tell the HangID annotation was removed in bug 784145 over 10 years ago so I say it's safe to get rid of them :-)

Flags: needinfo?(gsvelto)

Thank you, Gabriele!

Grabbing this to remove them.

Assignee: nobody → willkg
Status: NEW → ASSIGNED

This went to production in bug #1763234. Marking as FIXED.

Status: ASSIGNED → RESOLVED
Closed: 10 months ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.