960326 - Disable debug B2G mochitests on mozilla-aurora

Status: RESOLVED FIXED

(Release Engineering :: General, defect)

Description

[Jonathan Griffin (:jgriffin)]

They're broken and not likely to be fixed; we should just turn them off.

Comment 1

[Jonathan Griffin (:jgriffin)]

Attachment: Disable debug mochitests on older branches,

I hope this is the right logic.

Comment 2

[Aki Sasaki (not active)]

Comment on attachment 8360776 [details] [diff] [review]
Disable debug mochitests on older branches,

It's close!
Looks like we're hitting the same for loop issue, so this only disables half the debug mochitests:

(bb08)deathduck:/src/jgriffin/buildbot-configs/bm67-tests1-linux [21:07:28] (default)
600$ diff ../../{clean,dirty}.dump | more
3505d3504
< b2g_emulator_vm mozilla-aurora debug test mochitest-debug-1 ScriptFactory
3507d3505
< b2g_emulator_vm mozilla-aurora debug test mochitest-debug-3 ScriptFactory
3509d3506
< b2g_emulator_vm mozilla-aurora debug test mochitest-debug-5 ScriptFactory
3511d3507
< b2g_emulator_vm mozilla-aurora debug test mochitest-debug-7 ScriptFactory
3513d3508
< b2g_emulator_vm mozilla-aurora debug test mochitest-debug-9 ScriptFactory
3515d3509
< b2g_emulator_vm mozilla-aurora debug test mochitest-debug-11 ScriptFactory
3517d3510
< b2g_emulator_vm mozilla-aurora debug test mochitest-debug-13 ScriptFactory
3519d3511
< b2g_emulator_vm mozilla-aurora debug test mochitest-debug-15 ScriptFactory

Comment 3

[Aki Sasaki (not active)]

Comment on attachment 8360776 [details] [diff] [review]
Disable debug mochitests on older branches,

>+# Disable debug emulator mochitests on older branches
>+OLD_BRANCHES = set([name for name, branch in items_before(BRANCHES, 'gecko_version', 29)])
>+for b in BRANCHES.keys():
>+    branch = BRANCHES[b]
>+    if b in OLD_BRANCHES:
>+        if nested_haskey(branch['platforms'], 'emulator', 'ubuntu64_vm-b2g-emulator'):
>+            slave_p = branch['platforms']['emulator']['ubuntu64_vm-b2g-emulator']

r=me if you replace these 3 lines:

>+            for i in slave_p['debug_unittest_suites']:
>+                if i[0].startswith('mochitest-debug'):
>+                    slave_p['debug_unittest_suites'].remove(i)

with

+            slave_p['debug_unittest_suites'] = [x for x in slave_p['debug_unittest_suites'] if not x[0].startswith('mochitest-debug')]

which is ugly but gets rid of all 15 chunks on aurora.

Comment 4

[Jonathan Griffin (:jgriffin)]

Whoops, forgot about that problem from last time.  Thanks!

Comment 5

[Jonathan Griffin (:jgriffin)]

https://hg.mozilla.org/build/buildbot-configs/rev/8de90139c437

Comment 6

[[e-mail] (:bhearsum)]

in production