525263 - add maemo test coverage for all branches + try

Status: RESOLVED DUPLICATE of bug 538517

(Release Engineering :: General, defect)

Description

[Aki Sasaki (not active)]

I think this is a q4 goal.

Comment 1

[Aki Sasaki (not active)]

John: for Try, could you make sure the sendchange specifies that the branch is "try" (or some variant)?  We need to send the info to graphs-stage instead of graphs.m.o and the tinderbox page should be MozillaTry instead of Mobile or Tracemonkey.

Comment 2

[Aki Sasaki (not active)]

probably needs to have 'maemo' in the branch too; don't want to trigger incoming winmo tests with maemo try builds. "maemo-try"?

Comment 3

[Aki Sasaki (not active)]

John: joduinn pointed out you should probably talk w/ lsblakk and bhearsum about the large Try refactor going on currently.

Comment 4

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

i have the *-mobile-master side work done I think.

Lukas, Ben,

what is the plan for mobile try builders?

Comment 5

[[e-mail] (:bhearsum)]

I don't think the first pass on try-as-a-branch is going to change how we do mobile builds on try. So, implement it whichever way is best for you. When we convert the try mobile stuff over to mobile_config.py/mobile_master.py we can deal with moving all of the mobile stuff over at once.

Comment 6

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

Attachment: mobile-try-configs-mobile-master.diff

These are the buildbot-configs changes that create a maemo-try branch.  It also adds a switch to enable/disable queue collapsing on unittests and talos runs.

This is not complete as I need to add to the build factory the commands to build the packaged tests and uploading them alongside the other build artifacts.  I have test runs that have been built as nightlies (and known to work) running through these factories right now to test that mobile-master side code works.

I have only made the changes on mobile-staging.  I will copy the changes over to mobile on an r+

With the location that builds are uploaded to, we need to ignore self signed certificates which is done in the soon to be accompanying buildbotcustom patch

Comment 7

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

Attachment: mobile-try-custom-mobile-master.diff

We need this to disable checking ssl certificates for package urls.

We will need to run: 
sed -i -e 's/wget/wget --no-check-certificate/' /tools/maemkit/hackTestUrl.sh
on all the slaves to make sure that the packaged test files get uploaded as well

Comment 8

[Aki Sasaki (not active)]

Comment on attachment 418244 [details] [diff] [review]
mobile-try-configs-mobile-master.diff

>+BRANCHES['mobile-try']['platforms']['n810']['talos_branch'] = 'maemo-try'

This should be MozillaTest:

FAIL: Failed to send data 5 times... quitting
RETURN:send failed, graph server says:
RETURN:No branch_id for a branch_name 'maemo-try' can be found

We also need mail notifications for Try.

I'd also like to see the mobile/ configs -- I know it's a pain maintaining both while in active development, but I want to make sure the tinderbox/graphserver/etc. bits are right.

Other than that I'm glad to see this is working =)

Comment 9

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

Yep, i have that change locally, i forgot to include it in the patch.

As discussed in IRC, is it ok for me to get the jobs working now and get the mail notifications as a follow up?

Comment 10

[Aki Sasaki (not active)]

I'd say put it all in if a) it's relatively quick (a few days?) and b) I weren't going on vacation soon, especially since this will probably tie up mobile devices with no one paying attention otherwise.  But I am, and it would be good to see these running, so sure.

Comment 11

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

All of the work for this bug is going to be done as part of the config rewrite in bug 538517.