Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(213)

Issue 11826019: Move sandbox_linux_unittests back to FYI bots (Closed)

Created:
7 years, 11 months ago by Isaac (away)
Modified:
7 years, 11 months ago
CC:
chromium-reviews, ilevy+watch_chromium.org, bulach+watch_chromium.org, yfriedman+watch_chromium.org, peter+watch_chromium.org, klundberg+watch_chromium.org, frankf+watch_chromium.org, nilesh
Visibility:
Public.

Description

Move sandbox_linux_unittests back to FYI bots 4 tests are failing to run. http://build.chromium.org/p/chromium.linux/builders/Android%20Tests%20%28dbg%29/builds/6093/steps/sandbox_linux_unittests/logs/stdio They were passing on FYI bot but kicking until we figure out why they are failing here. BUG=166704 TBR=jln Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=175711

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -2 lines) Patch
M build/android/buildbot/bb_tests.py View 1 chunk +1 line, -1 line 0 comments Download
M build/android/run_tests.py View 1 chunk +0 lines, -1 line 0 comments Download

Messages

Total messages: 10 (0 generated)
Isaac (away)
TBR: jln. +Nilesh: Do you know why we aren't reporting a failure when tests FAIL_TO_RUN?
7 years, 11 months ago (2013-01-09 05:12:47 UTC) #1
jln (very slow on Chromium)
On 2013/01/09 05:12:47, Isaac wrote: > TBR: jln. > > +Nilesh: Do you know why ...
7 years, 11 months ago (2013-01-09 08:33:11 UTC) #2
jln (very slow on Chromium)
On 2013/01/09 08:33:11, Julien Tinnes wrote: > On 2013/01/09 05:12:47, Isaac wrote: > > TBR: ...
7 years, 11 months ago (2013-01-09 08:33:31 UTC) #3
Isaac (away)
Doubtful. Same tests failed a several builds in a row. Could be a bad phone ...
7 years, 11 months ago (2013-01-09 08:39:57 UTC) #4
jln (very slow on Chromium)
On 2013/01/09 08:39:57, Isaac wrote: > Doubtful. Same tests failed a several builds in a ...
7 years, 11 months ago (2013-01-09 08:41:24 UTC) #5
Isaac (away)
We encountered an android script issue where it wasn't complaining about tests timing out. We're ...
7 years, 11 months ago (2013-01-09 19:42:54 UTC) #6
frankf
crbug.com/169064 On 2013/01/09 19:42:54, Isaac wrote: > We encountered an android script issue where it ...
7 years, 11 months ago (2013-01-09 20:22:56 UTC) #7
jln (DO NOT USE THIS)
Timeout in these tests is surprising. Do you remember which specific tests it was? The ...
7 years, 11 months ago (2013-01-09 21:12:33 UTC) #8
nilesh
From: http://build.chromium.org/p/chromium.linux/builders/Android%20Tests%20%28dbg%29/builds/6093/steps/sandbox_linux_unittests/logs/stdio The timed out test was BrokerProcess.OpenOpenFilePermsNoClientCheck The timeout in this test caused other ...
7 years, 11 months ago (2013-01-09 21:38:45 UTC) #9
jln (DO NOT USE THIS)
7 years, 11 months ago (2013-01-09 22:21:09 UTC) #10
Thanks!

I'm very surprised this could fail, there is no Android specific code
around there (and it seems to work fine on the FYI bot). Do you have
any idea what could be printing these messages: "Fifo created..." it
looks as if there was some hooks on socketpair().

If anyone has a physical device that looks more like the main WF bot
than the FYI bots, that would be handy as well.

Julien

On Wed, Jan 9, 2013 at 9:38 PM, Nilesh Agrawal
<nileshagrawal@chromium.org> wrote:
> From:
>
http://build.chromium.org/p/chromium.linux/builders/Android%20Tests%20%28dbg%...
>
> The timed out test was BrokerProcess.OpenOpenFilePermsNoClientCheck
> The timeout in this test caused other tests in this particular shard to not
> run.
>
>
> On Wed, Jan 9, 2013 at 1:12 PM, Julien Tinnes <jln@google.com> wrote:
>>
>> Timeout in these tests is surprising. Do you remember which specific tests
>> it was? The link you posted no longer works.
>>
>> Looking at the FYI test right now, everything seems to run fine. I don't
>> see any test not being accounted for.
>>
>>
>> On Wed, Jan 9, 2013 at 8:22 PM, <frankf@chromium.org> wrote:
>>>
>>> crbug.com/169064
>>>
>>>
>>> On 2013/01/09 19:42:54, Isaac wrote:
>>>>
>>>> We encountered an android script issue where it wasn't complaining about
>>>> tests
>>>> timing out.  We're going to fix that issue and see if the sandbox unit
>>>> tests
>>>> stay green on the fyi bots.  In the meantime, you could look into the
>>>> timeout
>>>> issue if you wanted or you could wait and see how fyi bots fair after
>>>> the
>>>> android script fix.
>>>
>>>
>>>
>>> https://chromiumcodereview.appspot.com/11826019/
>>
>>
>

Powered by Google App Engine
This is Rietveld 408576698