Closed
Bug 1018377
Opened 11 years ago
Closed 11 years ago
Update trychooser for Android 2.3
Categories
(Release Engineering :: General, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: gbrown, Assigned: gbrown)
Details
(Keywords: trychooser)
Attachments
(2 files)
7.36 KB,
patch
|
emorley
:
review+
|
Details | Diff | Splinter Review |
4.37 KB,
patch
|
emorley
:
review+
|
Details | Diff | Splinter Review |
trychooser is missing some test jobs introduced for Android 2.3, such as plain-reftest-9 ... plain-reftest-12.
Updated•11 years ago
|
Assignee: gbrown → kmoir
![]() |
Assignee | |
Comment 1•11 years ago
|
||
I opted to change "Android 2.2" to "Android 2.3" rather than "Android 2.2/2.3", in hopes of a rapid removal of 2.2.
New tests for Android 2.3:
plain-reftest-9
plain-reftest-10
plain-reftest-11
plain-reftest-12
xpcshell-1
xpcshell-2
xpcshell-3
jsreftest-4, jsreftest-5, and jsreftest-6 were previously marked x86 only, but are not currently running for Android x86. Now they *are* needed for Android 2.3, so I removed the "only x86" comments.
Similarly, reftest-9, reftest-10, crashtest-1, crashtest-2, and crashtest-3 were x86 only, but none of them are running for Android x86. crashtest-1 and crashtest-2 *are* needed for Android 2.3, so I removed the "only x86" comments and removed crashtest-3.
There were "not x86" comments for robocop-3, robocop-4, and robocop-5, but robocop-3 is actually valid for Android x86. I opted to remove the 2 remaining valid x86 comments: If we try to distinguish which jobs are valid on x86, we should do the same for 2.3 vs 4.0, and I think that is just too complicated.
Finally, I noticed that "xpcshell" was in both the desktop section and the Android-only section, so I removed it from the Android-only section.
Attachment #8431808 -
Flags: review?(emorley)
Updated•11 years ago
|
Assignee: kmoir → gbrown
Comment 2•11 years ago
|
||
Comment on attachment 8431808 [details] [diff] [review]
update trychooser for Android 2.3 and Android x86
Review of attachment 8431808 [details] [diff] [review]:
-----------------------------------------------------------------
lgtm :-)
Attachment #8431808 -
Flags: review?(emorley) → review+
Updated•11 years ago
|
Status: NEW → ASSIGNED
![]() |
Assignee | |
Comment 3•11 years ago
|
||
![]() |
Assignee | |
Comment 4•11 years ago
|
||
In bug 1023314, we add more chunks for Android 2.3 and B2G reftests:
- there are now 16 Android 2.3 reftest chunks
- on next reconfig there will be 20 B2G reftest chunks
Attachment #8439308 -
Flags: review?(emorley)
Comment 5•11 years ago
|
||
Comment on attachment 8439308 [details] [diff] [review]
add more entries for more Android/B2G reftest chunks
Yey for the day when this is generated from a central json manifest :-)
Attachment #8439308 -
Flags: review?(emorley) → review+
![]() |
Assignee | |
Comment 6•11 years ago
|
||
Comment 7•11 years ago
|
||
Is this ready to be deployed then?
![]() |
Assignee | |
Comment 8•11 years ago
|
||
Yes!
Comment 9•11 years ago
|
||
trychooser change is now live - please verify at http://trychooser.pub.build.mozilla.org/
![]() |
Assignee | |
Comment 10•11 years ago
|
||
That looks fine. Thanks much.
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Updated•8 years ago
|
Component: Tools → General
You need to log in
before you can comment on or make changes to this bug.
Description
•