Closed
Bug 669108
Opened 13 years ago
Closed 13 years ago
Public crash reports should display TotalVirtualMemory, AvailableVirtualMemory, SystemMemoryUsePercentage, AvailablePageFile, and AvailablePhysicalMemory
Categories
(Socorro :: Webapp, task)
Socorro
Webapp
Tracking
(Not tracked)
VERIFIED
FIXED
2.4.2
People
(Reporter: jwatt, Assigned: laura)
References
Details
Attachments
(1 file)
(deleted),
image/png
|
Details |
Right now to get information on the memory use (the fields TotalVirtualMemory, AvailableVirtualMemory and SystemMemoryUsePercentage...on Mac at least) at the time of a crash you need someone with admin access to crash-stats.mozilla.com to get that for you. This info should just be available on the public reports.
If someone can point me to the parts of the code that need patched I'd be happy to have a go at fixing this.
Comment 1•13 years ago
|
||
There must be a reason, probably privacy or security concerns, why we hide those right now. Is there such a risk with that data? How large is it?
Comment 2•13 years ago
|
||
I don't believe there's any reason, there's no privacy risk to exposing this data AFAICT. I think we just treat all data from the client as private-by-default unless we explicitly make it public in the Socorro code.
Updated•13 years ago
|
Component: Socorro → General
Product: Webtools → Socorro
Assignee | ||
Updated•13 years ago
|
Component: General → Webapp
Comment 3•13 years ago
|
||
Also please publicly report the field "OOMAllocationSize", which I just added in bug 716638. It will not be present in all reports.
Comment 4•13 years ago
|
||
Laura, this looks relatively easy and would be a big win for a bunch of people looking at our OOM crashes and bugs, is it possible to get this prioritized?
Comment 5•13 years ago
|
||
While we're piling on here, it would be great to get AvailablePageFile and AvailablePhysicalMemory (bug 720444) added as well.
Updated•13 years ago
|
Summary: Public crash reports should display TotalVirtualMemory, AvailableVirtualMemory and SystemMemoryUsePercentage → Public crash reports should display TotalVirtualMemory, AvailableVirtualMemory, SystemMemoryUsePercentage, AvailablePageFile, and AvailablePhysicalMemory
Comment 6•13 years ago
|
||
Unless we're concerned about fingerprinting on Available{PhysicalMemory,PageFile}? It seems that you'd be able to fingerprint much better with e.g. the list of add-ons.
Assignee | ||
Comment 7•13 years ago
|
||
Lars, I don't think we have those in the processed json so you'll need to make a processor change first.
Assignee: nobody → lars
Target Milestone: --- → 2.4.2
Assignee | ||
Comment 8•13 years ago
|
||
We need to get access to the the raw json in the UI to solve this.
Assignee: lars → laura
Assignee | ||
Comment 9•13 years ago
|
||
pull request https://github.com/mozilla/socorro/pull/333
Comment 10•13 years ago
|
||
Commits pushed to master at https://github.com/mozilla/socorro
https://github.com/mozilla/socorro/commit/8735569798967b0b91898ee7b4b8e3ab542aa63b
Bug 669108 - forgot OOMAllocationSize added here
https://github.com/mozilla/socorro/commit/49b4bdad598df7ff9a899c9d3cc3abfbcaf289f1
Merge pull request #333 from lauraxt/669108
Fixes bug 669108, add some more stuff from the raw JSON to the UI. Also added better error checking.
Updated•13 years ago
|
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Comment 11•13 years ago
|
||
Thank you mightily :rhelmer for submitting a crash to stage for verification purposes.
https://crash-stats.allizom.org/report/index/8331e757-0eee-43ba-9ad8-037c52120210
Bumping to QA verified on stage. The following fields are no present on crash reports:
TotalVirtualMemory
AvailableVirtualMemory
SystemMemoryUsePercentage
AvailablePageFile
AvailablePhysicalMemory
OOMAllocationSize
Updated•13 years ago
|
Status: RESOLVED → VERIFIED
Comment 12•13 years ago
|
||
Yes, TotalVirtualMemory, AvailableVirtualMemory, SystemMemoryUsePerce are now visible on crash reports.
Comment 13•13 years ago
|
||
We should probably get a followup to format these better, or at least add units. (They're all bytes, except for System Memory Use Percentage, which is obviously a percentage.)
Updated•13 years ago
|
You need to log in
before you can comment on or make changes to this bug.
Description
•