Closed
Bug 1056973
Opened 10 years ago
Closed 10 years ago
Schedule web-platform-tests on linux64 opt on trunk branches
Categories
(Release Engineering :: General, defect)
Release Engineering
General
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: jgriffin, Assigned: jgriffin)
References
Details
Attachments
(2 files)
3.19 KB,
patch
|
nthomas
:
review+
RyanVM
:
checked-in-
|
Details | Diff | Splinter Review |
979 bytes,
patch
|
jgriffin
:
review+
RyanVM
:
checked-in-
|
Details | Diff | Splinter Review |
We're going to schedule web-platform-tests on linux64 opt on trunk branches. We've been aiming towards this goal for some time, but this has taken on new urgency to support Media Source Extension tests for ajones.
The tests are currently running on cedar and seem to be green, but we can initially hide them if that proves to be untrue.
Assignee | ||
Comment 1•10 years ago
|
||
Passes test-masters.sh
Attachment #8476884 -
Flags: review?(jlund)
Comment 2•10 years ago
|
||
Comment on attachment 8476884 [details] [diff] [review]
Schedule wpt on linux64 opt on trunk branches,
Review of attachment 8476884 [details] [diff] [review]:
-----------------------------------------------------------------
Apologies for the delay. I am on PTO atm until wed. If this is left to me, I may look at it before then but I can't guarantee :)
Assignee | ||
Comment 3•10 years ago
|
||
Comment on attachment 8476884 [details] [diff] [review]
Schedule wpt on linux64 opt on trunk branches,
Redirecting...
Attachment #8476884 -
Flags: review?(jlund) → review?(nthomas)
Comment 4•10 years ago
|
||
Comment on attachment 8476884 [details] [diff] [review]
Schedule wpt on linux64 opt on trunk branches,
Review of attachment 8476884 [details] [diff] [review]:
-----------------------------------------------------------------
::: mozilla-tests/config.py
@@ +1917,4 @@
> if slave_platform not in BRANCHES['cedar']['platforms'][platform]:
> continue
>
> + BRANCHES['cedar']['platforms'][platform][slave_platform]['opt_unittest_suites'] += WEB_PLATFORM_REFTESTS[:]
Looks like this will take away the four opt chunks on cedar, is that expected ? r+ either way (as is, or appending WEB_PLATFORM_TESTS_CHUNKED[:] too).
Attachment #8476884 -
Flags: review?(nthomas) → review+
Assignee | ||
Comment 5•10 years ago
|
||
> Looks like this will take away the four opt chunks on cedar, is that expected ? r+ either way
> (as is, or appending WEB_PLATFORM_TESTS_CHUNKED[:] too).
It should be handled now as part of the general
for name, branch in items_at_least(BRANCHES, 'gecko_version', 34):
block, so I just removed the special-casing for cedar.
Assignee | ||
Comment 6•10 years ago
|
||
Comment on attachment 8476884 [details] [diff] [review]
Schedule wpt on linux64 opt on trunk branches,
https://hg.mozilla.org/build/buildbot-configs/rev/fd2a3190eeb4
Attachment #8476884 -
Flags: checked-in+
Comment 7•10 years ago
|
||
FWIW, I dumped the builder list to see that.
Assignee | ||
Comment 8•10 years ago
|
||
Assignee | ||
Comment 9•10 years ago
|
||
Comment on attachment 8478641 [details] [diff] [review]
Re-enable wpt-chunked on cedar for non-linux64 platforms,
r+ from nthomas via bug 1041535.
Attachment #8478641 -
Flags: review+
Assignee | ||
Comment 10•10 years ago
|
||
Comment on attachment 8478641 [details] [diff] [review]
Re-enable wpt-chunked on cedar for non-linux64 platforms,
https://hg.mozilla.org/build/buildbot-configs/rev/2d4f9bd49195
Attachment #8478641 -
Flags: checked-in+
Comment 11•10 years ago
|
||
This landed too early; the wpt code itself hasn't landed yet, so these tests never make it past "pending".
Comment 12•10 years ago
|
||
Comment on attachment 8476884 [details] [diff] [review]
Schedule wpt on linux64 opt on trunk branches,
Reverted both patches.
https://hg.mozilla.org/build/buildbot-configs/rev/532b2f69daf4
Attachment #8476884 -
Flags: checked-in+ → checked-in-
Updated•10 years ago
|
Attachment #8478641 -
Flags: checked-in+ → checked-in-
Comment 13•10 years ago
|
||
Rolled out to prod with reconfig on 2014-08-26 08:21 PT
Assignee | ||
Comment 14•10 years ago
|
||
Whoops, sorry for pulling the trigger too quickly.
Comment 15•10 years ago
|
||
Assignee | ||
Updated•10 years ago
|
Status: NEW → RESOLVED
Closed: 10 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
•