New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 715784 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Apr 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 1
Type: Bug



Sign in to add a comment

Browser_tests not running on the bots

Project Member Reported by mmenke@chromium.org, Apr 26 2017

Issue description

I made a CL to remove proxy resolution, and see if browser_tests passed with it - and while I did run into failures, no browser tests failed.  Looking more closely, though, browser_tests didn't even run on any of the trybots.  To make absolutely sure this was the case, I added a CHECK(false) to a browser_tests, and it did not fail.

CL:  https://codereview.chromium.org/2841163002/ (Note that there are a few test failures in 2 or test fixtures, but none of them are browser_tests).  In particular, look at the Linux and OSX tests:  http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_rel_ng/builds/439858 and https://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_ng/builds/439515

I think this counts as a recipes issue?
 

Comment 1 by estaab@chromium.org, Apr 26 2017

Cc: dpranke@chromium.org phajdan.jr@chromium.org
Components: Infra>Client>Chrome
Cc: hinoka@chromium.org andyb...@chromium.org
Labels: Infra-Troopers
Owner: dpranke@chromium.org
Status: Assigned (was: Untriaged)
I'm not 100% sure what's going on here, but I'm guessing/hoping it's a side effect of  bug 714336 , and in particular machenbach's "partial revert" from this morning, https://codereview.chromium.org/2842963003, which I think might've regressed things.

I have a CL in the CQ that will hopefully fix  bug 714336 , so let's see if that lands and clears things up.

Comment 3 by mmenke@chromium.org, Apr 26 2017

Sorry, I did search for browser_tests bugs in the tracker, but somehow overlooked that one, when skimming the list.  I'll try again tomorrow, once that CL has landed.
The CL just landed; I'll trigger a couple try jobs on your patch and we can see what happens.

Comment 6 by mmenke@chromium.org, Apr 27 2017

That's great, thanks!

Sign in to add a comment