Closed Bug 1272541 Opened 7 years ago Closed 7 years ago

Add more data to SystemResourceMonitor.as_dict()

Categories

(Release Engineering :: Applications: MozharnessCore, defect)

defect
Not set
normal

Tracking

(firefox49 fixed)

RESOLVED FIXED
Tracking Status
firefox49 --- fixed

People

(Reporter: gps, Assigned: gps)

References

Details

Attachments

(2 files)

Patch coming shortly.
We add some system information including processor count
and memory sizes. We also add an "overall" section describing
total resource usage. This (surprisingly) wasn't defined.

This commit is the first in a series to reconcile the differences
between the JSON format in mozsystemmonitor and what
`mach build` writes so we can write a single tool to visualize
the data.

Review commit: https://reviewboard.mozilla.org/r/52381/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/52381/
Attachment #8752015 - Flags: review?(ahalberstadt)
Attachment #8752016 - Flags: review?(ahalberstadt)
Comment on attachment 8752015 [details]
MozReview Request: Bug 1272541 - Expose more data from SystemResourceMonitor.as_dict; r?ahal

https://reviewboard.mozilla.org/r/52381/#review49483
Attachment #8752015 - Flags: review?(ahalberstadt) → review+
Attachment #8752016 - Flags: review?(ahalberstadt) → review+
Comment on attachment 8752016 [details]
MozReview Request: Bug 1272541 - Bump mozsystemmonitor to version 0.2; r?ahal

https://reviewboard.mozilla.org/r/52383/#review49485
Blocks: 1272768
https://hg.mozilla.org/mozilla-central/rev/a13f0e8cfa3f
https://hg.mozilla.org/mozilla-central/rev/8b019a092a67
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.