1053916 - Enable chunk-by-dir for mochitest-plain on Android

Status: RESOLVED WORKSFORME

(Testing :: Mochitest, defect)

Description

[Ryan VanderMeulen [:RyanVM]]

I know we've discussed doing this before, but AFAICT we never filed a bug for it. We've been seeing a lot of heisenoranges lately that appear to be related to random changes in chunking as tests are added/removed/moved. Given that chunk-by-dir is already the default for desktop, we should enable it for mobile as well.

Comment 1

[Ryan VanderMeulen [:RyanVM]]

https://tbpl.mozilla.org/?tree=Try&rev=65431fa9128b

Comment 2

[Ryan VanderMeulen [:RyanVM]]

The end result of that Try run was massively-imbalanced chunks, leading to some pushing 2+hr runtime and others timing out. Clearly we'll need something better.

Comment 3

[Ryan VanderMeulen [:RyanVM]]

Might be interesting to play with this more once more of content/ and dom/ get flattened.

Comment 4

[Andrew Halberstadt [:ahal]]

(In reply to Ryan VanderMeulen [:RyanVM UTC-4] from comment #2)
> The end result of that Try run was massively-imbalanced chunks, leading to
> some pushing 2+hr runtime and others timing out. Clearly we'll need
> something better.

I had a 'chunk-by-manifest' idea that might help. But rather than hi-jack this bug I filed bug 1054247 to discuss it.

Comment 5

[Ryan VanderMeulen [:RyanVM]]

https://treeherder.mozilla.org/ui/#/jobs?repo=try&revision=4c61763f7464

--chunk-by-dir on Android appears to be broken at the moment. I've filed bug 1089771 to track getting that fixed. On B2G, things look mostly-OK, except we're still timing out on debug. I may try another run with the dom-level* tests re-enabled as well to see if they change the chunking around enough to make it work. Otherwise, I'm not sure what options we're going to have until we have smarter chunking options.

Comment 6

[Ryan VanderMeulen [:RyanVM]]

We're way past this nowadays.