Bug 95980

Summary:

Introduce the Chromium Android Release (Tests) bot

Product:

WebKit

Reporter:

Peter Beverloo <peter>

Component:

New Bugs

Assignee:

Peter Beverloo <peter>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

abarth, lforschler, mrowe, wsiegrist

Priority:

Version:

528+ (Nightly build)

Hardware:

Unspecified

OS:

Unspecified

Bug Depends on:

96113

Bug Blocks:

84843, 84845

Attachments:

Description	Flags
Patch	none

Peter Beverloo

Reported 2012-09-06 07:15:38 PDT

Introduce the Chromium Android Release (Tests) bot

Attachments
Patch (2.49 KB, patch) 2012-09-06 07:17 PDT, Peter Beverloo	no flags	Details Formatted Diff Diff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.

Peter Beverloo

Comment 1 2012-09-06 07:17:26 PDT

Created attachment 162502 [details] Patch

Peter Beverloo

Comment 2 2012-09-06 07:18:47 PDT

The machine is ready to be attached. Bill or Lucas, I'll also be needing a password to connect the machine to the master. It'll be listed as the second bot in the Chromium category.

Peter Beverloo

Comment 3 2012-09-06 08:18:48 PDT

Some initial numbers running this locally: The RunChromiumWebKitUnitTests (webkit_unit_tests) and RunUnitTests (TestWebKitAPI) steps will fail, because they haven't been updated yet to support running tests on devices. Support for this will be added next week. A full layout test cycle, spread over four devices, currently takes slightly more than an hour. Out of 32291 found tests, 24118 will be ran, 1855 of which fail right now. There are 28 unexpected passes, 15 text failures and 1756 image failures. This *probably* is due to an image color issue, as can be seen in these two images: http://peter.sh/files/absolute-poition-in-rtl-parent-actual.png http://peter.sh/files/absolute-poition-in-rtl-parent-expected.png (trivia: these two are the first renders made for Android using only upstream WebKit and Chromium code, early July.) Running TestWebKitAPI locally currently has one failure (bug 95748). Running webkit_unit_tests locally currently crashes because certain files are not available, which will be addressed in the runner code. In our branch it passes 1116 tests, meaning 36 have been disabled. This will of course be addressed :-).

Adam Barth

Comment 4 2012-09-06 09:30:49 PDT

Comment on attachment 162502 [details] Patch Cool!

William Siegrist

Comment 5 2012-09-06 15:19:03 PDT

Slave password generated and sent to Peter.

Peter Beverloo

Comment 6 2012-09-07 02:13:36 PDT

(In reply to comment #5) > Slave password generated and sent to Peter. Thank you! I'm landing this.

Peter Beverloo

Comment 7 2012-09-07 02:19:44 PDT

Committed r127847: <http://trac.webkit.org/changeset/127847>

William Siegrist

Comment 8 2012-09-07 08:25:34 PDT

Reconfig failed: + service buildbot checkconfig Checking buildbot config: /var/buildbot/master.cfg:792: DeprecationWarning: Passing a BuildStep subclass to factory.addStep is deprecated. Please pass a BuildStep instance instead. Support will be dropped in v0.8.7. self.addStep(RunEflAPITests) Configuration Errors: builder 'Chromium Android Release (Tests)' uses unknown slaves u'google-android-tests'

Peter Beverloo

Comment 9 2012-09-07 08:32:07 PDT

Sorry about that! Patch uploaded here: https://bugs.webkit.org/show_bug.cgi?id=96113

Peter Beverloo

Comment 10 2012-09-07 09:06:46 PDT

Fixed in http://trac.webkit.org/changeset/127875

Note You need to log in before you can comment on or make changes to this bug.