969210 - make test-perf fail to collect mem stats

Status: RESOLVED FIXED

(Firefox OS Graveyard :: Gaia::PerformanceTest, defect, P1)

Description

[Hubert Figuiere [:hub]]

Since very recently, collecting memory stats on buri/hamachi files.

b2g-info output starts with "Expected to read 4 fields from fscanf(/proc/2942//stat), but got 2." which is not expected. I can't figure why but we could make the meminfo.js code deal with it better.

Comment 1

[Hubert Figuiere [:hub]]

Ok, this is caused by not having this revision of gonk-misc on my buri:

ac1b69cc36ce8cf64c8ce585826e7a940b349072

I think the simple solution is to actually update it on my device.

Comment 2

[Hubert Figuiere [:hub]]

Attachment: Pull request

I think that even though I need to update b2g-info on my device, we can still have an error message pop out. Make this more robust.

Comment 3

[Julien Wajsberg [:julienw]]

Comment on attachment 8372054 [details] [review]
Pull request

I agree with the reasoning, but I think we can do a more reliable function. See github :)

Comment 4

[Hubert Figuiere [:hub]]

Comment on attachment 8372054 [details] [review]
Pull request

Updated the pull request.

Comment 5

[Julien Wajsberg [:julienw]]

Comment on attachment 8372054 [details] [review]
Pull request

r=me with nits for the commit "Rewrite the parser to be more permissive and foresee more changes."

Comment 6

[Hubert Figuiere [:hub]]

Nits addressed. Commit
https://github.com/mozilla-b2g/gaia/commit/b2be8fdf1f22ec3d4e4fbcdf8a8c0cbbabdb8718

Merge
https://github.com/mozilla-b2g/gaia/commit/f1471587ce4f6d773b78665f1db6a35e32aa63b1

Comment 7

[Hubert Figuiere [:hub]]

v1.3t uplift
https://github.com/mozilla-b2g/gaia/commit/7de8902dca508248d6da9627e9592ba57df5294d