Closed
Bug 1268622
Opened 8 years ago
Closed 8 years ago
Talos should specify whether a run is "e10s" as a special property of the suite
Categories
(Testing :: Talos, defect)
Testing
Talos
Tracking
(e10s+, firefox49 fixed)
RESOLVED
FIXED
mozilla49
People
(Reporter: wlach, Assigned: rwood)
References
(Blocks 1 open bug)
Details
(Whiteboard: [talos_wishlist])
Attachments
(1 file)
Right now perfherder categorizes talos data as being "e10s" (i.e. the test was run with e10s enabled) by looking at its job symbol, but this is a bit of a hack: https://github.com/mozilla/treeherder/blob/e7e98d0155e17198b85a85af2bcb9eaa40d9a44b/treeherder/etl/perf.py#L35 Instead, it should specify `extraOptions: ["e10s"]` on the suite level of the PERFHERDER_DATA object it spits out. Once this is done, we can remove yet another special case from inside Perfherder itself. It's been a while since I've touched Talos, but I suspect this might be a good first bug for someone to pick up.
Updated•8 years ago
|
Whiteboard: [talos_wishlist]
Comment 1•8 years ago
|
||
wlach, what about pgo?
Reporter | ||
Comment 2•8 years ago
|
||
(In reply to Joel Maher (:jmaher) from comment #1) > wlach, what about pgo? That's handled via the (separate) option collection property in perfherder already.
Updated•8 years ago
|
Blocks: e10s-tests
tracking-e10s:
--- → +
Assignee | ||
Updated•8 years ago
|
Assignee: nobody → rwood
Status: NEW → ASSIGNED
Assignee | ||
Comment 3•8 years ago
|
||
Review commit: https://reviewboard.mozilla.org/r/51711/diff/#index_header See other reviews: https://reviewboard.mozilla.org/r/51711/
Attachment #8750956 -
Flags: review?(jmaher)
Comment 4•8 years ago
|
||
Comment on attachment 8750956 [details] MozReview Request: Bug 1268622 - Talos should specify whether a run is e10s as a special property of the suite; r?jmaher https://reviewboard.mozilla.org/r/51711/#review48761 do this adjust the perfherder_data output structure at all? If so, I will be happy to r+ this.
Attachment #8750956 -
Flags: review?(jmaher)
Reporter | ||
Comment 5•8 years ago
|
||
(In reply to Joel Maher (:jmaher) from comment #4) > Comment on attachment 8750956 [details] > MozReview Request: Bug 1268622 - Talos should specify whether a run is e10s > as a special property of the suite; r?jmaher > > https://reviewboard.mozilla.org/r/51711/#review48761 > > do this adjust the perfherder_data output structure at all? If so, I will > be happy to r+ this. It appears to do the right thing: https://treeherder.mozilla.org/logviewer.html#?job_id=20637872&repo=try#L1374 Note for the future: Please run patches like this by me explicitly, since incorrectly formatted output has the potential to cause problems (it has in the past).
Comment 6•8 years ago
|
||
Comment on attachment 8750956 [details] MozReview Request: Bug 1268622 - Talos should specify whether a run is e10s as a special property of the suite; r?jmaher https://reviewboard.mozilla.org/r/51711/#review48787 thanks :wlach for clarifying. It is hard to remember all the right places to adjust or ask for additional reviews. I should have know to run this by your :wlach, between rwood and myself we can do better in the future.
Attachment #8750956 -
Flags: review+
Assignee | ||
Comment 7•8 years ago
|
||
https://reviewboard.mozilla.org/r/51711/#review48761 Yes, see "extraOptions": ["e10s"] for each suite: https://treeherder.mozilla.org/logviewer.html#?job_id=20637870&repo=try#L1405 If the test run is non-e10s then the output structure is not changed, for example: https://treeherder.mozilla.org/logviewer.html#?job_id=20637876&repo=try#L1359 The same output modification is reflected in the local.json when running locally i.e. --develop mode.
Comment 9•8 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/8a6a980d3953
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
status-firefox49:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla49
You need to log in
before you can comment on or make changes to this bug.
Description
•