606310 - Enable n900 unittest/talos results on TryServer

Status: RESOLVED FIXED

(Release Engineering :: General, defect, P4)

Description

[John O'Duinn [:joduinn] (please use "needinfo?" flag)]

Blocked on getting additional phones online, to handle the load.

Comment 1

[John Ford [:jhford] CET/CEST Berlin Time]

Attachment: buildbot-configs v1

This patch adds the try server branch.  This branch doesn't allow merging of builds and doesn't have any polling.

Comment 2

[Aki Sasaki (not active)]

Comment on attachment 488565 [details] [diff] [review]
buildbot-configs v1

Hm, does your .hgrc contain

[diff]
git=true
showfunc = 1
unified = 9

?

r=me if s,try_mode,enable_try,g

Comment 3

[John Ford [:jhford] CET/CEST Berlin Time]

[diff]
git=True
showfunc=True

i will land with that change

Comment 4

[Aki Sasaki (not active)]

(I had to apply the patch to see context for

@@ -107,6 +107,7 @@ for branch_name in sorted(BRANCHES.keys(
                 'slavebuilddir': '/builds/slave_build_dir',
                 'factory': talos_factory,
                 'category': bb_branch,
+                'mergeRequests': not branch.get('try_mode', False),

; that would have been fixed by unified = 9.)

Comment 5

[John Ford [:jhford] CET/CEST Berlin Time]

Attachment: buildbotcustom v1

this patch adds a sendchange to the n900-gtk builds.

Comment 6

[John Ford [:jhford] CET/CEST Berlin Time]

Attachment: buildbotcustom

minor typo.

Comment 7

[John Ford [:jhford] CET/CEST Berlin Time]

Attachment: buildbotcustom v2

works on a staging try server,

Comment 8

[Aki Sasaki (not active)]

Comment on attachment 488966 [details] [diff] [review]
buildbotcustom v2

This is pretty ugly, but is a workable solution until we get make upload working for maemo.

To sum up our discussion, the long term goals are:

* make upload for maemo, with appropriate sendchanges as everything else (blocked by multilocale)
* either get the n900s on an existing 08x test master that is already part of the sendchange list, or add pmm/smm to a config list of sendchange hosts
* maemo tests -> mozilla-tests/

and the short term followup bugs are:

* mail notification for maemo try tests

r=me to get this working asap without blocking on the above.

Comment 9

[John Ford [:jhford] CET/CEST Berlin Time]

Comment on attachment 488565 [details] [diff] [review]
buildbot-configs v1

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

Comment 10

[John Ford [:jhford] CET/CEST Berlin Time]

Comment on attachment 488966 [details] [diff] [review]
buildbotcustom v2

http://hg.mozilla.org/build/buildbotcustom/rev/25fae1f9a41e

Comment 11

[John Ford [:jhford] CET/CEST Berlin Time]

Attachment: send email

this patch adds the ability to send email on for n900s on try.  We should probably make sure that only known-good tests are enabled otherwise we will mail-bomb try users.

Comment 12

[John Ford [:jhford] CET/CEST Berlin Time]

Attachment: only run green tests on tryserver

This patch removes broken tests from the test suites to run on mozilla central.

Verified that the code only shows working suites by adding 

print 'Running these things for tryserver:'
print '\n'.join(
          BRANCHES['tryserver']['platforms']['n900-gtk']['test_suites'].keys() +
          BRANCHES['tryserver']['platforms']['n900-gtk']['talos_suites'].keys()
)

to the end of the config-production-n900.py file.  I got the output:

Running these things for tryserver:
tpan
twinopen
tsspider
ts
tsvg
tgfx
tdhtml

I need to run this in staging.  Ugly patch though.

Comment 14

[John Ford [:jhford] CET/CEST Berlin Time]

Comment on attachment 489367 [details] [diff] [review]
only run green tests on tryserver

works in staging.

Comment 15

[Aki Sasaki (not active)]

Comment on attachment 489367 [details] [diff] [review]
only run green tests on tryserver

That looks right.

Comment 16

[John Ford [:jhford] CET/CEST Berlin Time]

Comment on attachment 489361 [details] [diff] [review]
send email

http://hg.mozilla.org/build/buildbot-configs/rev/2196e8c24b97

Comment 17

[John Ford [:jhford] CET/CEST Berlin Time]

Comment on attachment 489367 [details] [diff] [review]
only run green tests on tryserver

https://bugzilla.mozilla.org/show_bug.cgi?id=606310

Comment 18

[John Ford [:jhford] CET/CEST Berlin Time]

all done here