Closed
Bug 815786
Opened 13 years ago
Closed 13 years ago
Disable testCheck3 in Talos
Categories
(Testing :: Talos, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
mozilla20
People
(Reporter: gbrown, Assigned: jmaher)
References
Details
Attachments
(4 files)
1.21 KB,
patch
|
Callek
:
review+
|
Details | Diff | Splinter Review |
999 bytes,
patch
|
k0scist
:
review+
|
Details | Diff | Splinter Review |
4.12 KB,
patch
|
gbrown
:
review+
|
Details | Diff | Splinter Review |
1.16 KB,
patch
|
emorley
:
review+
|
Details | Diff | Splinter Review |
Android Talos test "testCheck3" (aka rck3, tcheck3, remote-trobocheck3) has evolved to be functionally equivalent to testCheck2.
The testCheck2 and testCheck3 have been producing virtually identical results for some time now:
http://graphs.mozilla.org/graph.html#tests=[[204,63,20],[201,63,20]]&sel=none&displayrange=7&datatype=running
We should stop running testCheck3 to save tbpl test time, and then I think there will be no reason to keep the testCheck3 code:
https://hg.mozilla.org/mozilla-central/file/ae2d36c5dc26/mobile/android/base/tests/testCheck3.java.in
Assignee | ||
Comment 1•13 years ago
|
||
this removes all instances of trobocheck3 from the buildbot-configs repo. I assume there is another definition for this somewhere, but I was unable to find it in buildbot-configs or buildbotcustom repositories.
Attachment #686890 -
Flags: review?(bugspam.Callek)
Assignee | ||
Comment 2•13 years ago
|
||
and the patches keep rolling in.
Assignee | ||
Comment 3•13 years ago
|
||
and 3 makes a trifecta.
Attachment #686895 -
Flags: review?(gbrown)
Updated•13 years ago
|
Attachment #686892 -
Flags: review?(jhammel) → review+
![]() |
Reporter | |
Comment 4•13 years ago
|
||
Comment on attachment 686895 [details] [diff] [review]
remove tcheck3 sources from mozilla-central repository (1.0)
Thanks!
Attachment #686895 -
Flags: review?(gbrown) → review+
Comment 5•13 years ago
|
||
Comment on attachment 686890 [details] [diff] [review]
remove trobocheck3 from the buildbot configs (1.0)
Review of attachment 686890 [details] [diff] [review]:
-----------------------------------------------------------------
We also need to/should update http://mxr.mozilla.org/build/source/tools/trychooser/index.html?force=1#150 Please do that with rs+=me and land it (I forget the process to deploying trychooser updates though)
Attachment #686890 -
Flags: review?(bugspam.Callek) → review+
Assignee | ||
Comment 6•13 years ago
|
||
http://hg.mozilla.org/build/buildbot-configs/rev/af8dcb18ed0c
I don't know the process for try chooser, that is a new venue for me to work in.
Assignee | ||
Comment 7•13 years ago
|
||
Attachment #687082 -
Flags: review?(edmorley.bz)
Updated•13 years ago
|
Attachment #687082 -
Flags: review?(edmorley.bz) → review+
Assignee | ||
Comment 8•13 years ago
|
||
landed in tools repo for try chooser:
https://hg.mozilla.org/build/tools/rev/8489dcda1573
Comment 9•13 years ago
|
||
This is in production.
Comment 10•13 years ago
|
||
The buildbotb part, I mean.
Assignee | ||
Comment 11•13 years ago
|
||
Assignee | ||
Comment 12•13 years ago
|
||
landed on talos:
http://hg.mozilla.org/build/talos/rev/0f264d4c4205
I will let this close out when the inbound patch makes it through to central. The rest is just cleanup at this point.
Comment 13•13 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla20
You need to log in
before you can comment on or make changes to this bug.
Description
•