WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
Bug 95980
Introduce the Chromium Android Release (Tests) bot
https://bugs.webkit.org/show_bug.cgi?id=95980
Summary
Introduce the Chromium Android Release (Tests) bot
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.
Top of Page
Format For Printing
XML
Clone This Bug