Closed
Bug 1330697
Opened 6 years ago
Closed 6 years ago
verify xperf_whitelist.json entries are still needed
Categories
(Testing :: Talos, defect)
Testing
Talos
Tracking
(firefox54 fixed)
RESOLVED
FIXED
mozilla54
Tracking | Status | |
---|---|---|
firefox54 | --- | fixed |
People
(Reporter: jmaher, Assigned: jmaher)
Details
Attachments
(1 file, 1 obsolete file)
6.58 KB,
patch
|
rwood
:
review+
|
Details | Diff | Splinter Review |
right now we have a long laundry list of files that we expect to be accessed on startup (measured by the windows xperf talos test): https://dxr.mozilla.org/mozilla-central/source/testing/talos/talos/xtalos/xperf_whitelist.json?q=path%3Axperf_whitelist.json&redirect_type=single#16 I suspect some of these files are not needed anymore, keeping this list up to date would be important. we also have mainthreadIO fileIO measurements in this file: https://dxr.mozilla.org/mozilla-central/source/testing/talos/talos/mtio-whitelist.json again, keeping that list up to date (i.e. removing old entries) seems important. as a note, sometimes things are intermittent or pgo only (bug 1324062, comment 79)
Assignee | ||
Comment 1•6 years ago
|
||
a meager attempt to do this for xperf_whitelist.json: https://treeherder.mozilla.org/#/jobs?repo=try&revision=8008b1bb89bc7a402728f460ea2382d7c3f62ac8
Assignee | ||
Comment 2•6 years ago
|
||
remove some obvious xperf_whitelist.json entries. a failure case here: https://treeherder.mozilla.org/#/jobs?repo=try&revision=8008b1bb89bc7a402728f460ea2382d7c3f62ac8 adding those missing ones back in here: https://treeherder.mozilla.org/#/jobs?repo=try&revision=20cc8bc16b8a9bdc55776affd8b3f6937af1fd9e
Comment 3•6 years ago
|
||
Comment on attachment 8826280 [details] [diff] [review] xperf_sanity.patch LGTM, R+ pending green on try
Attachment #8826280 -
Flags: review?(rwood) → review+
Pushed by jmaher@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/e8ae38b46276 remove old xperf_whitelist.json entries. r=rwood
Assignee | ||
Comment 5•6 years ago
|
||
I had mentioned mainthread_io, I had removed almost all the entries in a try server push and we are not detecting anything- possibly this isn't running anymore.
Assignee | ||
Comment 6•6 years ago
|
||
as a note, I removed what I thought would be easy to remove, possibly other files could be removed, although I believe it would be better served in the future as an annual cleanup.
I had to back this out because some Windows pgo xperf runs failed with it in: https://treeherder.mozilla.org/logviewer.html#?job_id=68825854&repo=mozilla-inbound https://hg.mozilla.org/integration/mozilla-inbound/rev/4ef05cc6af4e
Flags: needinfo?(jmaher)
Assignee | ||
Comment 8•6 years ago
|
||
thanks for backing this out, I am testing a patch on pgo and non pgo to validate all is well. it looks like a small chunk of my removals are needed.
Flags: needinfo?(jmaher)
Assignee | ||
Comment 9•6 years ago
|
||
lets try this again, now I made sure to test on pgo before calling this good. Removed some old entries, put things in a more logical order, and added some new entries (for intermittent issues)
Attachment #8826280 -
Attachment is obsolete: true
Attachment #8829207 -
Flags: review?(rwood)
Comment 10•6 years ago
|
||
Comment on attachment 8829207 [details] [diff] [review] cleanup old xperf whitelist file LGTM
Attachment #8829207 -
Flags: review?(rwood) → review+
Comment 11•6 years ago
|
||
Pushed by jmaher@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/e0f2dbc7c3cb verify xperf_whitelist.json entries are still needed. r=rwood
Comment 12•6 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/e0f2dbc7c3cb
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
status-firefox54:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla54
You need to log in
before you can comment on or make changes to this bug.
Description
•