Mochitest devtools dt9 chunk frequently last to finish on linux64 debug because of its size

RESOLVED FIXED in Firefox 51

Status

Testing
Mochitest
RESOLVED FIXED
11 months ago
10 months ago

People

(Reporter: wlach, Assigned: jgriffin)

Tracking

(Blocks: 1 bug)

unspecified
mozilla51
Points:
---

Firefox Tracking Flags

(firefox51 fixed)

Details

MozReview Requests

Submitter Diff Changes Open Issues Last Updated
Loading...
Error loading review requests:

Attachments

(1 attachment)

Looking at my infraherder prototype (http://wlach.github.io/treeherder/ui/infra.html#/last-finished), it seems like the mochitest dt9 chunk is frequently the last to finish on Linux debug.

Looking at this example:  https://treeherder.mozilla.org/#/jobs?repo=try&revision=89fccfd367377df5296d16f41b9933217e8c63f2

dt1: 44 minute(s)
dt2: 56 minute(s)
dt3: 53 minute(s)
dt4: 26 minute(s)
dt5: 63 minute(s)
dt6: 33 minute(s)
dt7: 49 minute(s)
dt8: 56 minute(s)
dt9: 83 minute(s)

It seems like reshuffling the chunks is in order. It looks like dt9 is tests in the `devtools/client/debugger/test/mochitest/` directory. :chmanchester suggested that we may need to split that up, since it's run-by-dir.

Updated

11 months ago
Blocks: 1278680
(Assignee)

Comment 1

11 months ago
According to ahal, chunk-by-runtime won't split manifests, so we might be able to fix this by splitting the tests in that dir into two (or more) manifests.

We should probably re-normalize the chunks as well; it looks like this suite hasn't been touched for 11 months: https://hg.mozilla.org/mozilla-central/filelog/tip/testing/runtimes/mochitest-devtools-chrome.runtimes.json
(Assignee)

Comment 2

10 months ago
Here's a try run with devtools/client/debugger/test/mochitest split into two manifests:

https://treeherder.mozilla.org/#/jobs?repo=try&revision=359e706d089c

Runtimes on linux64 debug for all chunks are: 18, 31, 27, 20, 18, 15, 12, 18, 26, 24

So this seems to have the desired effect.
Comment hidden (mozreview-request)
(Assignee)

Updated

10 months ago
Assignee: nobody → jgriffin

Comment 4

10 months ago
mozreview-review
Comment on attachment 8784397 [details]
Bug 1294489 - Split directory into two manifests for better chunking,

https://reviewboard.mozilla.org/r/73860/#review73562

Seems fine to me (sorry, was at a conference and PTO recently). It's worth noting that we are about to land a new debugger and working on an entirely new test suite, so in the next month or two we can retire these old debugger tests.
Attachment #8784397 - Flags: review?(jlong) → review+
Comment hidden (mozreview-request)

Comment 6

10 months ago
Pushed by jgriffin@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/d819b0431209
Split directory into two manifests for better chunking, r=jlongster

Comment 7

10 months ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/d819b0431209
Status: NEW → RESOLVED
Last Resolved: 10 months ago
status-firefox51: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla51
You need to log in before you can comment on or make changes to this bug.