Closed
Bug 918463
Opened 11 years ago
Closed 11 years ago
update talos configs to reflect updated tests
Categories
(Release Engineering :: General, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: jmaher, Assigned: jmaher)
References
Details
Attachments
(3 files)
3.13 KB,
patch
|
kmoir
:
review+
|
Details | Diff | Splinter Review |
4.08 KB,
patch
|
armenzg
:
review+
|
Details | Diff | Splinter Review |
28.48 KB,
patch
|
Details | Diff | Splinter Review |
we need to switch tsvg -> tsvgx
we need to switch ts -> ts_paint
we need to make sure these are on aurora (after the merge)
Assignee | ||
Comment 1•11 years ago
|
||
Comment 2•11 years ago
|
||
Comment on attachment 807369 [details] [diff] [review]
update talos mobile configs (1.0)
lgtm I tested the patch on my staging master
Attachment #807369 -
Flags: review?(kmoir) → review+
Assignee | ||
Comment 3•11 years ago
|
||
landed this:
https://hg.mozilla.org/build/buildbot-configs/rev/c79c55ed0764
this will get picked up in the next reconfig.
Comment 6•11 years ago
|
||
I'm trying to figure out what we want to do in here.
jmaher, please correct me if I got this wrong.
Why is tcanvasmark in the patch?
I believe these are the changes (IIUC) we have to make:
- Common flags
- enable tsvgx & ts_paint by default ('enable_by_default': True)
- disable tsvg & ts by default ('enable_by_default': False)
- Make these changes on mozilla-beta (FF25), mozilla-release (FF24)
- disable tsvgx & ts_paint
+BRANCHES['mozilla-beta']['remote-tsvgx_tests'] = (0, False, TALOS_REMOTE_FENNEC_OPTS, ANDROID_NOT_MOZPOOL)
+BRANCHES['mozilla-beta']['remote-ts_paint_tests'] = (0, False, TALOS_REMOTE_FENNEC_OPTS, ANDROID_NOT_MOZPOOL)
+BRANCHES['mozilla-release']['remote-tsvgx_tests'] = (0, False, TALOS_REMOTE_FENNEC_OPTS, ANDROID_NOT_MOZPOOL)
+BRANCHES['mozilla-release']['remote-ts_paint_tests'] = (0, False, TALOS_REMOTE_FENNEC_OPTS, ANDROID_NOT_MOZPOOL)
- enable tsvg & ts
+BRANCHES['mozilla-beta']['remote-ts_tests'] = (1, False, TALOS_REMOTE_FENNEC_OPTS, ANDROID_NOT_MOZPOOL)
+BRANCHES['mozilla-beta']['remote-tsvg_tests'] = (1, False, TALOS_REMOTE_FENNEC_OPTS, ANDROID_NOT_MOZPOOL)
+BRANCHES['mozilla-release']['remote-ts_tests'] = (1, False, TALOS_REMOTE_FENNEC_OPTS, ANDROID_NOT_MOZPOOL)
+BRANCHES['mozilla-release']['remote-tsvg_tests'] = (1, False, TALOS_REMOTE_FENNEC_OPTS, ANDROID_NOT_MOZPOOL)
NOTE: We don't run mobile testing on ESR releases
Right now I see these [1]:
- tsvgx enabled for m-c & m-a
- ts_paint enabled nowhere
- tsvg enabled for FF27 branches (except m-c), m-b & m-r
- ts enabled FF27 branches (except m-c), m-b, m-r
[1] ./setup-master.py master bm10-tests1-tegra && ~/repos/releng/braindump/buildbot-related/builder_list.py master.cfg > builder_list.txt
Assignee | ||
Comment 7•11 years ago
|
||
ok, Canvasmark was added to aurora as it is firefox 26+, I am not sure of the best way to make tests follow the release trains.
let me work on a patch to enable the tests by default, turn them off on m-r, m-b and then leave the old ones on only for m-r/m-b.
Assignee | ||
Comment 8•11 years ago
|
||
Attachment #809137 -
Flags: review?(armenzg)
Comment 9•11 years ago
|
||
This is the diff of the old builders versus the new ones (with this patch applied).
This lists are produced with builder_list.py from the braindump repo.
Comment 10•11 years ago
|
||
Comment on attachment 809137 [details] [diff] [review]
updated mobile configs to ride the trains (1.0)
Review of attachment 809137 [details] [diff] [review]:
-----------------------------------------------------------------
r+ conditional on adding a "mozilla-release" block similar to the "mozilla-beta" block.
Everything else is perfect.
Attachment #809137 -
Flags: review?(armenzg) → review+
Assignee | ||
Comment 11•11 years ago
|
||
landed:
https://hg.mozilla.org/build/buildbot-configs/rev/18a48374b3cd
this can be picked up in the next reconfig, no rush for this specific patch.
Comment 12•11 years ago
|
||
In production
Assignee | ||
Updated•11 years ago
|
Status: ASSIGNED → 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
•