Closed
Bug 1252292
Opened 10 years ago
Closed 9 years ago
Set up ash for e10s testing
Categories
(Release Engineering :: General, defect)
Release Engineering
General
Tracking
(e10s+)
RESOLVED
FIXED
| Tracking | Status | |
|---|---|---|
| e10s | + | --- |
People
(Reporter: jgriffin, Assigned: RyanVM)
References
(Blocks 1 open bug)
Details
Attachments
(5 files)
|
6.88 KB,
patch
|
rail
:
review+
RyanVM
:
checked-in+
|
Details | Diff | Splinter Review |
|
77.96 KB,
text/plain
|
Details | |
|
46.90 KB,
text/plain
|
Details | |
|
807 bytes,
patch
|
rail
:
review+
RyanVM
:
checked-in+
|
Details | Diff | Splinter Review |
|
3.08 KB,
patch
|
rail
:
review+
catlee
:
checked-in+
|
Details | Diff | Splinter Review |
We're going to set up e10s testing on all platforms on a project branch, since we don't have the capacity to add e10s tests on all trunk branches.
Ash looks like a good candidate; it hasn't been used since Dec 4.
| Reporter | ||
Comment 1•10 years ago
|
||
Nick, are you ok repurposing ash for this?
Flags: needinfo?(nthomas)
Comment 2•10 years ago
|
||
Yes, I forgot to relinquish it when I was done. It's been transferred over to you on the wiki.
Flags: needinfo?(nthomas)
Updated•10 years ago
|
Blocks: e10s-tests
tracking-e10s:
--- → +
| Assignee | ||
Comment 3•10 years ago
|
||
I've got a mostly-working patch for this.
Assignee: nobody → ryanvm
| Assignee | ||
Comment 4•10 years ago
|
||
After a ridiculous amount of studying differences.txt and allthethings_with_patch.json, I'm finally convinced that I've got this all right from a logic standpoint. Please do grade me on style too, though :)
Attachment #8730032 -
Flags: review?(rail)
| Assignee | ||
Comment 5•10 years ago
|
||
| Assignee | ||
Comment 6•10 years ago
|
||
Comment 7•10 years ago
|
||
Comment on attachment 8730032 [details] [diff] [review]
schedule only e10s tests across all desktop platforms on Ash
Review of attachment 8730032 [details] [diff] [review]:
-----------------------------------------------------------------
lgtm
Attachment #8730032 -
Flags: review?(rail) → review+
| Assignee | ||
Comment 8•10 years ago
|
||
Comment on attachment 8730032 [details] [diff] [review]
schedule only e10s tests across all desktop platforms on Ash
http://hg.mozilla.org/build/buildbot-configs/rev/8b6119cfc6a9
Attachment #8730032 -
Flags: checked-in+
| Assignee | ||
Comment 9•10 years ago
|
||
(In reply to Ryan VanderMeulen [:RyanVM] from comment #8)
> http://hg.mozilla.org/build/buildbot-configs/rev/8b6119cfc6a9
In production. Leaving the bug open until the other needed work is finished, though.
| Assignee | ||
Comment 10•10 years ago
|
||
Ash is being considered production from a test validity and importance standpoint, so let's not leave its jobs sitting in line behind everything else.
Attachment #8730356 -
Flags: review?(rail)
Updated•10 years ago
|
Attachment #8730356 -
Flags: review?(rail) → review+
| Assignee | ||
Comment 11•10 years ago
|
||
Comment on attachment 8730356 [details] [diff] [review]
set Ash to priority 3 to match mozilla-central
https://hg.mozilla.org/build/buildbot-configs/rev/50c1fdd4ff3e
Attachment #8730356 -
Flags: checked-in+
Comment 12•10 years ago
|
||
Comment 13•10 years ago
|
||
Comment 14•10 years ago
|
||
MozReview-Commit-ID: A7qVu8AofOC
Updated•10 years ago
|
Attachment #8732117 -
Flags: review?(rail)
Comment 15•10 years ago
|
||
I'm pretty sure this code (using `branch` from a previous loop) introduced the intermittent checkconfig failures we were seeing this week.
Updated•10 years ago
|
Attachment #8732117 -
Flags: review?(rail) → review+
Comment 16•10 years ago
|
||
https://hg.mozilla.org/build/buildbot-configs/rev/abea5b637aa4ba2753f58ed741018e84f2504d82
Bug 1252292 - Fix ash e10s processing r=rail
Updated•10 years ago
|
Attachment #8732117 -
Flags: checked-in+
| Assignee | ||
Updated•9 years ago
|
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
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
•