linux_site_isolation bot is purple |
||||
Issue descriptionhttps://build.chromium.org/p/tryserver.chromium.linux/builders/linux_site_isolation?numbuilds=200 which blocks CQ when "CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation" gets included. browser_tests and content_browsertests seem to not be able to start. Perhaps related to https://codereview.chromium.org/2459813003 ?
,
Nov 2 2016
Thanks for looking into this Alex and helping identify the culprit. git blame says that both lines referring to testing/buildbot/filters/isolate-extensions.*.filter come from r428848, so let me just try to revert this CL to unblock linux_site_isolation bot on the CQ (the CL being reverted was only trying to switch the Windows bot from --site-per-process to --isolate-extensions testing - reverting it shouldn't be a problem AFAICT). FWIW, the revert is churning through the tryjobs at https://codereview.chromium.org/2466933003.
,
Nov 2 2016
FWIW, the linux_site_isolation tryjob for the revert is @ http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_site_isolation/builds/4437
,
Nov 2 2016
I'm guessing the purpleness itself is caused by the fact that we add site-per-process filters as a dependency so that they're picked up for swarming, but we don't do that for isolate-extensions filters [1]. So the bot wasn't able to find the isolate-extension filter file once it was unexpectedly switched to --isolate-extensions, and the revert should hopefully fix it. That means before relanding r428848, we should similarly add isolate-extensions.*.filter to deps, since swarming is now enabled on all our bots. [1] E.g., https://cs.chromium.org/chromium/src/chrome/test/BUILD.gn?q=site-per-process.*.filter&sq=package:chromium&l=2018&dr=C
,
Nov 2 2016
The purpleness is fixed starting in build https://build.chromium.org/p/chromium.fyi/builders/Site%20Isolation%20Linux/builds/12293 Unfortunately the purple has changed to red because of a new layout test. I've opened issue 661725 to track this. I believe the new failure shouldn't impact CQ because the test will fail with and without a patch. |
||||
►
Sign in to add a comment |
||||
Comment 1 by scottmg@chromium.org
, Nov 2 2016