commit | 159461b27b6a1696ac1df083cca61e07fed27989 | [log] [tgz] |
---|---|---|
author | Sven Zheng <svenzheng@chromium.org> | Fri May 06 03:54:39 2022 |
committer | Copybara-Service <copybara-worker@google.com> | Fri May 06 04:14:24 2022 |
tree | 13bb300f8556215655bdf67c3c3af1a7fbc9f898 | |
parent | 49f1169b8ffb1953cbd7021ef3847f5039818547 [diff] |
Revert "Experiment 5% run Lacros related tests on linux-chromeos-rel" This reverts commit 91374e2ba2d994c96183d64b2aa870cebe8dee0e. Reason for revert: Start having issues on CQ. So enable on CI back first allowing me debug further. I locally tested. With CI gn args tests can pass. But with CQ gn args, tests are crashed. Original change's description: > Experiment 5% run Lacros related tests on linux-chromeos-rel > > This will build Lacros without experimental on the builder. > The test suite will run at 5% experiment. > The builder resource was approved in crbug.com/1233066. > The test resource was approved in crbug.com/1287287. > Flakiness comparison: > browser_tests: https://data.corp.google.com/sites/chrome_generic_flakiness_dashboard_datasite/flake_report/?f=variant_builder:in:linux-chromeos-rel&f=realm:in:chromium:ci&f=variant_test_suite:in:browser_tests > browser_tests_require_lacros: https://data.corp.google.com/sites/chrome_generic_flakiness_dashboard_datasite/flake_report/?f=variant_builder:in:linux-chromeos-rel&f=realm:in:chromium:ci&f=variant_test_suite:in:browser_tests_require_lacros > This target has higher percentage of flake tests, but the > absolute number of flake tests is much lower. > > Bug: 1287287 > Change-Id: Id6899f271177b01e69e383a8206ff69db4d48c3a > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3597209 > Reviewed-by: Gary Tong <gatong@chromium.org> > Commit-Queue: Sven Zheng <svenzheng@chromium.org> > Cr-Commit-Position: refs/heads/main@{#994856} Bug: 1287287 Change-Id: I467d6fd02780395bfccaec99c1f63a93eee3cbad Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3630503 Auto-Submit: Sven Zheng <svenzheng@chromium.org> Reviewed-by: Gary Tong <gatong@chromium.org> Commit-Queue: Gary Tong <gatong@chromium.org> Cr-Commit-Position: refs/heads/main@{#1000238} NOKEYCHECK=True GitOrigin-RevId: 2ee638e14a7fc7232a004a59fa19fb83b7b0d78e
MB is a simple wrapper around the GN build tool.
It was originally written as part of the GYP->GN migration, in order to provide a uniform interface to either GYP or GN, such that users and bots can call one script and not need to worry about whether a given bot was meant to use GN or GYP.
It eventually grew additional functionality and is now still used even though everything is GN-only.
It supports two main functions:
“gen” - the main gn gen
invocation that generates the Ninja files needed for the build.
“analyze” - the step that takes a list of modified files and a list of desired targets and reports which targets will need to be rebuilt.
We also use MB as a forcing function to collect all of the different build configurations that we actually support for Chromium builds into one place, in //tools/mb/mb_config.pyl
.
For more information, see: