Closed
Bug 1112779
Opened 10 years ago
Closed 10 years ago
Turn off Mulet mochitests on all trees except Try and Cedar
Categories
(Release Engineering :: General, defect)
Release Engineering
General
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: RyanVM, Assigned: armenzg)
References
Details
Attachments
(2 files, 3 obsolete files)
2.43 KB,
patch
|
rail
:
review+
armenzg
:
checked-in+
|
Details | Diff | Splinter Review |
2.90 KB,
text/plain
|
RyanVM
:
feedback+
|
Details |
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.
Reporter | ||
Updated•10 years ago
|
Summary: Turn off Mulet mochitest-2/4 and reftests on all platforms → Turn off Mulet mochitest-2/4 and reftests on all trees
Assignee | ||
Comment 1•10 years ago
|
||
For the record, mochitests are also not running the right thing (bug 1072443).
Blocks: firefox-mulet
Reporter | ||
Comment 2•10 years ago
|
||
I'm honestly considering just completely disabling them in that case.
Comment 3•10 years ago
|
||
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.
Comment 4•10 years ago
|
||
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?
Comment 5•10 years ago
|
||
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.
Comment 6•10 years ago
|
||
Reftest are only enable on Try and Cedar, right?
Assignee | ||
Updated•10 years ago
|
Assignee: nobody → armenzg
Assignee | ||
Comment 7•10 years ago
|
||
Attachment #8538764 -
Flags: review?(rail)
Assignee | ||
Comment 8•10 years ago
|
||
Builders differing after this patch.
We only run reftests on Try and Cedar.
Attachment #8538766 -
Flags: feedback?(ryanvm)
Reporter | ||
Comment 9•10 years ago
|
||
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-
Reporter | ||
Updated•10 years ago
|
Summary: Turn off Mulet mochitest-2/4 and reftests on all trees → Turn off Mulet mochitests on all trees except Try and Cedar
Assignee | ||
Comment 10•10 years ago
|
||
Attachment #8538764 -
Attachment is obsolete: true
Attachment #8538764 -
Flags: review?(rail)
Attachment #8538805 -
Flags: review?(rail)
Assignee | ||
Comment 11•10 years ago
|
||
Attachment #8538766 -
Attachment is obsolete: true
Attachment #8538806 -
Flags: feedback?(ryanvm)
Reporter | ||
Comment 12•10 years ago
|
||
Comment on attachment 8538806 [details]
differences.txt
Looks the same as before.
Attachment #8538806 -
Flags: feedback?(ryanvm) → feedback-
Updated•10 years ago
|
Attachment #8538805 -
Flags: review?(rail) → review+
Assignee | ||
Comment 13•10 years ago
|
||
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+
Assignee | ||
Comment 14•10 years ago
|
||
Attachment #8538806 -
Attachment is obsolete: true
Attachment #8540312 -
Flags: feedback?(ryanvm)
Assignee | ||
Comment 15•10 years ago
|
||
I believe this is live for the last 8 hours.
http://hg.mozilla.org/build/buildbot-configs/graph
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Reporter | ||
Comment 16•10 years ago
|
||
Comment on attachment 8540312 [details]
differences.txt
Love it.
Attachment #8540312 -
Flags: feedback?(ryanvm) → feedback+
Updated•7 years ago
|
Component: General Automation → General
You need to log in
before you can comment on or make changes to this bug.
Description
•