844989 - Replace fedora-b2g slaves with ubuntu64-b2g for b2g emulator test

Status: RESOLVED FIXED

(Release Engineering :: General, defect)

Description

[Rail Aliiev [:rail]]

Attachment: replace fedora

ATM the plan is to replace MOCHITEST + MARIONETTE + XPCSHELL for opt and probably debug builds.

The patch replaces fedora slaves for all branches except b2g18 and b2g18_v1_0_1 (we can discuss this list).

Comment 1

[Andrew Halberstadt [:ahal]]

Was just about to file this :)

The patch to disable the failing mochitests just landed on inbound and I was planning to land it on b2g18 after it gets merged to m-c. That means this patch is pretty much ready to land as soon as it gets reviewed.

Comment 2

[Rail Aliiev [:rail]]

Comment on attachment 718013 [details] [diff] [review]
replace fedora

The patch generates a sane diff for b2g_config.py dump and passed test-masters.sh

Comment 3

[Rail Aliiev [:rail]]

Comment on attachment 718013 [details] [diff] [review]
replace fedora

Review of attachment 718013 [details] [diff] [review]:
-----------------------------------------------------------------

::: mozilla-tests/b2g_config.py
@@ +688,3 @@
>  for branch in set(BRANCHES.keys()) - set(['cedar']):
> +    if branch in NON_UBUNTU_BRANCHES:
> +        print "removing", branch

I'll remove the line above, of course.

Comment 4

[Aki Sasaki (not active)]

Comment on attachment 718013 [details] [diff] [review]
replace fedora

r=me without the print, if the builder_list diff looks good.

Comment 5

[Rail Aliiev [:rail]]

Comment on attachment 718013 [details] [diff] [review]
replace fedora

I found some issues with the builder names...

Comment 6

[Rail Aliiev [:rail]]

Attachment: replace fedora

I found a problem (finally). The patch was trying to to mangle a mutable variable (slave_platforms.remove(...)). Using list(PLATFORMS[platform]['slave_platforms']) fixes it.

I also simplified the "evil" loop because b2g_config.py is simpler than config.py and we already have all ubuntu runable tests listed in platform_config (which can be redefined in branch_config).

Thanks for the tip, builder_list.py has a sane output now.

Comment 7

[Aki Sasaki (not active)]

Comment on attachment 718235 [details] [diff] [review]
replace fedora

I'm hoping that these for loops in both b2g_config.py and config.py will go away when we finish the switch over to ubuntu unittests for all suites that we can ?

Comment 8

[Rail Aliiev [:rail]]

ahal, do you think that we are ready to go and switch from Fedora to Ubuntu?

Comment 9

[Andrew Halberstadt [:ahal]]

Yep, let's ship it!

Comment 10

[Rail Aliiev [:rail]]

Comment on attachment 718235 [details] [diff] [review]
replace fedora

http://hg.mozilla.org/build/buildbot-configs/rev/8ef764230c91

Comment 11

[Rail Aliiev [:rail]]

in production

Comment 12

[Rail Aliiev [:rail]]

https://tbpl.mozilla.org/?tree=Mozilla-Inbound&rev=7bdc94c723f3

B2G Arm (VM) opt M(1 2 3 4 5 6 7 8 9) Mn X

All green.

Kudos (read beer) should go to ahal! :)

Comment 13

[Jonathan Griffin (:jgriffin)]

Awesome, many thanks to all involved!