Closed Bug 1112779 Opened 10 years ago Closed 9 years ago

Turn off Mulet mochitests on all trees except Try and Cedar

Categories

(Release Engineering :: General, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: RyanVM, Assigned: armenzg)

References

Details

Attachments

(2 files, 3 obsolete files)

Mochitest-2 and 4 were enabled in production on August 1. From the start, they were permafail and hidden immediately (see bug 1047571 and bug 1047572). In the following 4.5 months, literally nothing has happened in the bugs on file.

Similarly, reftests were enabled for Try only in early October and immediately hidden (see bug 1080123). Same story.

Instead of wasting resources indefinitely, we should just shut them off until someone's going to step up to actually get them fit to run in production.
Summary: Turn off Mulet mochitest-2/4 and reftests on all platforms → Turn off Mulet mochitest-2/4 and reftests on all trees
For the record, mochitests are also not running the right thing (bug 1072443).
I'm honestly considering just completely disabling them in that case.
It's not the case that they provide no value.  Rather, they're running in the parent process, and we'd prefer to have them running in a child process, like we do with b2gdesktop.  That said, it's very unclear what priority Mulet has right now; there's not been enough developer engagement to actually replace b2gdesktop, which was the original goal.
We still have some goals to replace b2g desktop with mulet during Q1.
We have been focusing on making gaia tests to work against it.
Until we have all tests suites up and running on CI, we can't officialy replace b2g desktop.
And until we replace it we won't get a lot of traction, so it is mostly me and gerard-majax trying to address whatever if broken.
The good news is that I heard from James that most tests and especially gaia one are already running on taskcluster!

Having said that, I'm not against disabling them as soon as it doesn't prevent us reenabling them once we are ready to switch to mulet (and fixed the failures)! Will it also prevent us from running them on Try?
Also, reftest is different than mochitests, gerard-majax has been working on greening them a lot recently.
Hopefully that's close to be green now.
Reftest are only enable on Try and Cedar, right?
Assignee: nobody → armenzg
Attachment #8538764 - Flags: review?(rail)
Attached file differences.txt (obsolete) —
Builders differing after this patch.

We only run reftests on Try and Cedar.
Attachment #8538766 - Flags: feedback?(ryanvm)
Comment on attachment 8538766 [details]
differences.txt

Per the IRC discussion w/ ochameau, we want to turn off *all* mochitests, not just 2 & 4.
Attachment #8538766 - Flags: feedback?(ryanvm) → feedback-
Summary: Turn off Mulet mochitest-2/4 and reftests on all trees → Turn off Mulet mochitests on all trees except Try and Cedar
Attachment #8538764 - Attachment is obsolete: true
Attachment #8538764 - Flags: review?(rail)
Attachment #8538805 - Flags: review?(rail)
Attached file differences.txt (obsolete) —
Attachment #8538766 - Attachment is obsolete: true
Attachment #8538806 - Flags: feedback?(ryanvm)
Comment on attachment 8538806 [details]
differences.txt

Looks the same as before.
Attachment #8538806 - Flags: feedback?(ryanvm) → feedback-
Attachment #8538805 - Flags: review?(rail) → review+
Comment on attachment 8538805 [details] [diff] [review]
[configs] Disable mulet mochitests except on Try/Cedar

My bad about attaching the wrong differences file. I will attach it again once I'm back to my Linux machine.
Attachment #8538805 - Flags: checked-in+
Attached file differences.txt
Attachment #8538806 - Attachment is obsolete: true
Attachment #8540312 - Flags: feedback?(ryanvm)
I believe this is live for the last 8 hours.
http://hg.mozilla.org/build/buildbot-configs/graph
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Comment on attachment 8540312 [details]
differences.txt

Love it.
Attachment #8540312 - Flags: feedback?(ryanvm) → feedback+
Blocks: 1129037
Component: General Automation → General
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: