Note: There are a few cases of duplicates in user autocompletion which are being worked on.

Please schedule desktop jit-tests on all trunk trees and make them ride the trains

RESOLVED FIXED

Status

Release Engineering
General Automation
RESOLVED FIXED
4 years ago
3 years ago

People

(Reporter: dminor, Assigned: kmoir)

Tracking

(Depends on: 1 bug)

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(4 attachments)

(Reporter)

Description

4 years ago
It looks like all of the consistent oranges for the desktop jit-tests on Cedar are now fixed and we're down to intermittents.

It is difficult to tell whether or not we hit more intermittents on the test machines than we do on the build machines while running on Cedar, due to the low volume of pushes.

I think we should turn these on for desktop builds on trunk trees, hidden for now, and see how things look.

We might need to resolve Bug 969986 first.
And three other unfiled blockers, besides getting all Linux to run on try and include them in trychooser:

* decide whether to run them on all Mac and Windows platforms (probably not)
* some parallel permaorange on 10.6 opt and WinXP opt+debug
* the Linux slaves are so brutally slow that for linux64 asan and debug, this'll tie with the very slowest of browser-chrome runs for being the long pole for completing tests on a push
Depends on: 969988, 969991
(Reporter)

Updated

4 years ago
Depends on: 973946
(Reporter)

Updated

4 years ago
Depends on: 906525
(Reporter)

Updated

3 years ago
Depends on: 977579
(Reporter)

Updated

3 years ago
No longer depends on: 973946
(Assignee)

Updated

3 years ago
Assignee: nobody → kmoir
(Reporter)

Updated

3 years ago
Depends on: 988522
(Reporter)

Updated

3 years ago
Blocks: 988532
dminor:  Can you address philor's questions in comment #1?  Should I indeed enable them on all trunk trees and ride the trains for all platforms or are there some platforms where they should be excluded for now?
Flags: needinfo?(dminor)
(Reporter)

Comment 3

3 years ago
I'm not sure if I'm the right person to decide this. I think the minimum would be to cover platforms that currently run them as part of make check (Linux, Windows XP, Mac OS X 10.8).

I have seen different results on Windows 7 and 8 than on WinXP on Cedar, but I'm not sure if we actually need that additional test coverage.

Ted, Terrence, any preferences here?
Flags: needinfo?(terrence)
Flags: needinfo?(ted)
Flags: needinfo?(dminor)
Let's schedule them on all platforms for now. If we decide we don't need them we can always selectively disable them later.
Flags: needinfo?(ted)
(although honestly I'd take philor's advice over mine)
Please do turn these on everywhere. I expect that this will expose real issues that are currently hidden because we are not already running them on all of our tier-1 platforms. We will fix anything that crops up.
Flags: needinfo?(terrence)
(Reporter)

Updated

3 years ago
No longer depends on: 906525
Created attachment 8400152 [details] [diff] [review]
bug973900.patch

Will attach builder diffs too
Attachment #8400152 - Flags: review?
(Assignee)

Updated

3 years ago
Attachment #8400152 - Flags: review? → review?(rail)
Created attachment 8400153 [details]
mac builder diff
Created attachment 8400155 [details]
linux builder diff
Created attachment 8400156 [details]
windows builder diff
Comment on attachment 8400152 [details] [diff] [review]
bug973900.patch

Review of attachment 8400152 [details] [diff] [review]:
-----------------------------------------------------------------

lgtm

::: mozilla-tests/config.py
@@ +1927,5 @@
> +                    if 'fedora' in slave_platform:
> +                        continue  # Don't use rev3 mini's with this stuff
> +                    BRANCHES[name]['platforms'][platform][slave_platform]['opt_unittest_suites'] += jittests[:]
> +                    BRANCHES[name]['platforms'][platform][slave_platform]['debug_unittest_suites'] += jittests[:]
> +     

A nit: can you remove the trailing spaces
Attachment #8400152 - Flags: review?(rail) → review+
(Assignee)

Updated

3 years ago
Attachment #8400152 - Flags: checked-in+
in production.
verified on tbpl
Status: NEW → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.