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

Issue 75493004: Spawn DBus by default as a workaround on all Linux bots. (Closed)

Created:
7 years, 1 month ago by Ken Russell (switch to Gerrit)
Modified:
7 years, 1 month ago
CC:
chromium-reviews, cmp-cc_chromium.org, Elliot Glaysher, oshima
Base URL:
https://chromium.googlesource.com/chromium/tools/build.git@master
Visibility:
Public.

Description

Spawn DBus by default as a workaround on all Linux bots. It seems likely that the intermittent hangs of browser_tests with Linux Aura have the same root cause as Issue 309093. Since that workaround worked well on the affected bots, enable it for all Linux bots. BUG=320131, 309093 Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=235869

Patch Set 1 #

Total comments: 2
Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -1 line) Patch
M scripts/slave/runtest.py View 1 chunk +1 line, -1 line 2 comments Download

Messages

Total messages: 6 (0 generated)
Ken Russell (switch to Gerrit)
Mike: please review. Thanks.
7 years, 1 month ago (2013-11-18 23:21:46 UTC) #1
ghost stip (do not use)
lgtm https://codereview.chromium.org/75493004/diff/1/scripts/slave/runtest.py File scripts/slave/runtest.py (left): https://codereview.chromium.org/75493004/diff/1/scripts/slave/runtest.py#oldcode1158 scripts/slave/runtest.py:1158: default=False, then, if this sticks we can remove ...
7 years, 1 month ago (2013-11-18 23:24:33 UTC) #2
ghost stip (do not use)
talk to the trooper and alert sheriffs on IRC before you deploy, just to be ...
7 years, 1 month ago (2013-11-18 23:25:26 UTC) #3
Ken Russell (switch to Gerrit)
https://codereview.chromium.org/75493004/diff/1/scripts/slave/runtest.py File scripts/slave/runtest.py (left): https://codereview.chromium.org/75493004/diff/1/scripts/slave/runtest.py#oldcode1158 scripts/slave/runtest.py:1158: default=False, On 2013/11/18 23:24:33, stip wrote: > then, if ...
7 years, 1 month ago (2013-11-18 23:37:30 UTC) #4
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/kbr@chromium.org/75493004/1
7 years, 1 month ago (2013-11-19 01:16:05 UTC) #5
commit-bot: I haz the power
7 years, 1 month ago (2013-11-19 01:17:13 UTC) #6
Message was sent while issue was closed.
Change committed as 235869

Powered by Google App Engine
This is Rietveld 408576698