Closed Bug 951223 Opened 6 years ago Closed 6 years ago

Schedule b2g desktop reftests on Cedar

Categories

(Release Engineering :: General, defect)

defect
Not set

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: ahal, Assigned: ahal)

References

Details

Attachments

(1 file)

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.
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 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+
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
In production.
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Component: General Automation → General
You need to log in before you can comment on or make changes to this bug.