Closed
Bug 951223
Opened 11 years ago
Closed 11 years ago
Schedule b2g desktop reftests on Cedar
Categories
(Release Engineering :: General, defect)
Release Engineering
General
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: ahal, Assigned: ahal)
References
Details
Attachments
(1 file)
(deleted),
patch
|
mozilla
:
review+
|
Details | Diff | Splinter Review |
The changes to the mozharness script landed, we should schedule them on Ash and Cedar. I still need to land gecko changes on both branches, but I can do that in parallel (or after) this bug.
Assignee | ||
Comment 1•11 years ago
|
||
I didn't realize Ash didn't have any exceptions anymore, so decided to just make do with cedar (shouldn't really need it on ash anyway).
I also noticed that the b2g desktop osx platform didn't have a suite config for mochitest (which is why it's failing on cedar), so I fixed that while I was at it. I can file a separate bug though if you'd prefer.
Attachment #8348859 -
Flags: review?(aki)
Comment 2•11 years ago
|
||
Comment on attachment 8348859 [details] [diff] [review]
Patch 1.0 - schedule b2g desktop reftests on cedar
Thanks!
I think we only really need to copy the list when it's by itself (e.g. BRANCHES[...] = TEST[:] ); when we concatenate lists we create a new one anyway.
I think it's only a minor perf hit, and isn't the only example of this; just noting.
The builder diffs look good:
(bb08)deathduck:/src/reviews [11:21:43]
540$ diff linux{clean,dirty}
3617a3618
> b2g_ubuntu64_vm cedar opt test reftest ScriptFactory
3620a3622
> b2g_ubuntu32_vm cedar opt test reftest ScriptFactory
(bb08)deathduck:/src/reviews [11:21:47]
541$ diff mac{clean,dirty}
1973a1974
> b2g_macosx64 cedar opt test reftest ScriptFactory
Attachment #8348859 -
Flags: review?(aki) → review+
Assignee | ||
Comment 3•11 years ago
|
||
Good point. I removed the slices:
https://hg.mozilla.org/build/buildbot-configs/rev/c329e84ccc56
Summary: Schedule b2g desktop reftests on Ash and Cedar → Schedule b2g desktop reftests on Cedar
Comment 4•11 years ago
|
||
In production.
Assignee | ||
Updated•11 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 11 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
•