Closed
Bug 1004791
Opened 11 years ago
Closed 11 years ago
Schedule Android 2.3 crashtests, js-reftests, plain reftests, and m-gl on all trunk trees and make them ride the trains
Categories
(Release Engineering :: General, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: gbrown, Assigned: kmoir)
References
Details
Attachments
(4 files)
(deleted),
patch
|
armenzg
:
review+
kmoir
:
checked-in+
|
Details | Diff | Splinter Review |
(deleted),
text/x-patch
|
Details | |
(deleted),
patch
|
armenzg
:
review+
kmoir
:
checked-in+
|
Details | Diff | Splinter Review |
(deleted),
text/plain
|
Details |
+++ This bug was initially created as a clone of Bug #989462 +++
Android 2.3 reftests are now running reliably on Ash and meet the requirements on https://wiki.mozilla.org/Sheriffing/Job_Visibility_Policy:
https://tbpl.mozilla.org/?tree=Ash&jobname=Android.*2.3.*
All of {C1 C2 C3 J1 J2 J3 J4 J5 J6 R1 R2 R3 R4 R5 R6 R7 R8 R9 R10 R11 R12} can now be run on trunk.
Mochitest-gl is the only remaining job on Ash that is not ready for trunk.
Android 2.3 crashtests, js-reftests, and plain-reftests must be run on ix slaves to avoid excessive run times and timeouts.
From my perspective, Android 2.3 mochitests, robocop, and xpcshell tests may continue running on aws slaves, or may also be run on ix slaves.
Reporter | ||
Comment 1•11 years ago
|
||
mochitest-gl is running reliably on ash now and can also be scheduled on trunk.
Summary: Schedule Android 2.3 crashtests, js-reftests, and plain reftests on all trunk trees and make them ride the trains → Schedule Android 2.3 crashtests, js-reftests, plain reftests, and m-gl on all trunk trees and make them ride the trains
Assignee | ||
Comment 2•11 years ago
|
||
patch to enable additional tests
gbrown: As a side note, in this patch I enabled these tests where gecko is at least 32. Previously there were limited tests on gecko31 i.e. limited tests on mozilla-aurora. However, to clean up our code so there weren't stanzas for disabled and enabled 2.3 tests, I just enabled all tests on 2.3 going forward where gecko is at least 32. This means that these tests will be disabled on m-a until the next merge. Let me know if this is okay, but I felt that it was important to clean up mobile_config.py. I'll attach a builder diff so you can see the difference.
Flags: needinfo?(gbrown)
Assignee | ||
Comment 3•11 years ago
|
||
builder diff
Reporter | ||
Comment 4•11 years ago
|
||
(In reply to Kim Moir [:kmoir] from comment #2)
> gbrown: As a side note, in this patch I enabled these tests where gecko is
> at least 32. Previously there were limited tests on gecko31 i.e. limited
> tests on mozilla-aurora. However, to clean up our code so there weren't
> stanzas for disabled and enabled 2.3 tests, I just enabled all tests on 2.3
> going forward where gecko is at least 32. This means that these tests will
> be disabled on m-a until the next merge. Let me know if this is okay, but I
> felt that it was important to clean up mobile_config.py. I'll attach a
> builder diff so you can see the difference.
That seems reasonable.
Of course there is interest in retiring the tegras as soon as possible, and I suppose we cannot retire them fully until these changes ride the trains and 32 is released. But I don't see a way around that: We only know that m-gl passes on 32, for example. It would probably take a lot of uplifts and re-testing to accelerate the tegra retirement.
Flags: needinfo?(gbrown)
Assignee | ||
Updated•11 years ago
|
Attachment #8421182 -
Flags: review?(armenzg)
Comment 5•11 years ago
|
||
Comment on attachment 8421182 [details] [diff] [review]
bug1004791.patch
Review of attachment 8421182 [details] [diff] [review]:
-----------------------------------------------------------------
This is great!
On another note, if armv6, armv7 and androidx86 are going to be tested on talos-linux64-ix we *might* need another round of iX purchases.
We might also have some suites that we can move to AWS without too much of a wall time hit compared to iX machines.
Attachment #8421182 -
Flags: review?(armenzg) → review+
Assignee | ||
Updated•11 years ago
|
Attachment #8421186 -
Flags: checked-in+
Assignee | ||
Updated•11 years ago
|
Attachment #8421186 -
Flags: checked-in+
Assignee | ||
Updated•11 years ago
|
Attachment #8421182 -
Flags: checked-in+
Comment 6•11 years ago
|
||
Live in production.
Assignee | ||
Comment 7•11 years ago
|
||
oops, they were not running on ix slaves, but rather ec2 instances which will lead to long test times. I've tested this on my master, will attach builder diff
Attachment #8421819 -
Flags: review?(armenzg)
Assignee | ||
Comment 8•11 years ago
|
||
builder diff
Updated•11 years ago
|
Attachment #8421819 -
Flags: review?(armenzg) → review+
Assignee | ||
Comment 9•11 years ago
|
||
Attachment #8421819 -
Flags: checked-in+
Assignee | ||
Comment 10•11 years ago
|
||
verified on tbpl
Status: NEW → 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
•