881623 - Build Architecture Info ended by | None in Crash Report

Status: RESOLVED FIXED

(Socorro :: General, task)

Description

[Scoobidiver (away)]

Build Architecture Info is always ended by | None in Crash Report.

See https://crash-stats-django.allizom.org/report/index/1fcdec5e-8fc7-404a-8622-babda2130605

Comment 1

[Robert Helmer [:rhelmer]]

Hmm from a quick look not sure where this is coming form - could it be the mware?

Comment 2

[[DEACTIVATED] Adrian Gaudebert]

That value comes straight from the processed crash, and the middleware doesn't do anything on it before sending it back. I suspect there is a problem in the processor, or in breakpad.

Comment 3

[Scoobidiver (away)]

(In reply to Adrian Gaudebert [:adrian] from comment #2)
> I suspect there is a problem in the processor, or in breakpad.
There's no issue in crash stats. See bp-1a3ab5b0-0a96-4e8a-98d3-3dd752130612.

Comment 4

[Robert Kaiser]

(In reply to Scoobidiver from comment #3)
> (In reply to Adrian Gaudebert [:adrian] from comment #2)
> > I suspect there is a problem in the processor, or in breakpad.
> There's no issue in crash stats. See bp-1a3ab5b0-0a96-4e8a-98d3-3dd752130612.

https://crash-stats-django.mozilla.org/report/index/1a3ab5b0-0a96-4e8a-98d3-3dd752130612 looks pretty much the same as the PHP version for this.

Comment 5

[Robert Kaiser]

(In reply to Robert Kaiser (:[e-mail]) [away until early June] from comment #4)
> https://crash-stats-django.mozilla.org/report/index/1a3ab5b0-0a96-4e8a-98d3-
> 3dd752130612 looks pretty much the same as the PHP version for this.

Sorry, I'm wrong, it's broken on django production as well.

Comment 6

[Robert Helmer [:rhelmer]]

OK this is definitely a processor bug - I looked in the processed JSON for the uuid in question and see:

"cpu_info": "GenuineIntel family 6 model 42 stepping 7 | None", 

Unblocking Django app. lars would you mind taking a look?

Comment 7

[Robert Kaiser]

Interesting, if this is a processor thing, why does the PHP UI not show the " | None" then?

Comment 8

[Scoobidiver (away)]

(In reply to Robert Kaiser (:[e-mail]) [away until early June] from comment #7)
> if this is a processor thing
I think an old version of processor software is running. See also bug 881720.

Comment 9

[Robert Kaiser]

The processor for the production installation is the same for both UIs (crash-stats.mozilla.com and crash-stats-django.mozilla.org), just like the rest of the backend.

Comment 10

[K Lars Lohn [:lars] [:klohn]]

this is, indeed, a bug in Processor2012.  It is corrected in https://github.com/mozilla/socorro/pull/1298

Comment 11

[[github robot]]

Commit pushed to master at https://github.com/mozilla/socorro

https://github.com/mozilla/socorro/commit/779d91c109b04317e82f479dbb6d9a370eecfffc
Merge pull request #1298 from twobraids/build-arch-info

Fixes Bug 881623 - Fixed bad call in getting number of CPUs from MDSW

Comment 12

[Scoobidiver (away)]

Socorro 52 should have landed yesterday.

Comment 13

[Robert Kaiser]

(In reply to Scoobidiver from comment #12)
> Socorro 52 should have landed yesterday.

No, it was delayed as 51 only was released this week.