| commit | 0629262ed3ce818e17816a9db0f500d7f8dddc7a | [log] [tgz] |
|---|---|---|
| author | Yusuf Mohsinally <mohsinally@google.com> | Fri May 20 17:38:23 2022 +0000 |
| committer | CQ Bot Account <flutter-scoped@luci-project-accounts.iam.gserviceaccount.com> | Fri May 20 17:38:23 2022 +0000 |
| tree | f7f44494a14c04385a8bc402c2bea67fd9f0d3a8 | |
| parent | 46af13d9f600611eec7376c90f79055333e96bc0 [diff] |
Revert "Removing staging linux VM tests from devicelab_staging_config.star" This reverts commit 2d7a0c1c6a6942d11b5a5ba9e666d87f0ecdd78c. Reason for revert: There is an issue with the cocoon scheduler thats preventing us from adding the staging builds. So relanding this to allow us to continue with our migration to Ubuntu. Original change's description: > Removing staging linux VM tests from devicelab_staging_config.star > > These tests are being moved to .ci.yaml, and no longer needed to > be run from devicelab_staging_config.star. > > Bug: 102406 > Change-Id: I113a20ef6c2821952941625d3ac5a2168d42f030 > Reviewed-on: https://flutter-review.googlesource.com/c/infra/+/29641 > Reviewed-by: Godofredo Contreras <godofredoc@google.com> > Reviewed-by: Casey Hillers <chillers@google.com> > Commit-Queue: Yusuf Mohsinally <mohsinally@google.com> # Not skipping CQ checks because original CL landed > 1 day ago. Bug: 102406 Change-Id: I93774792fde842115b413615c606661af6199a3d Reviewed-on: https://flutter-review.googlesource.com/c/infra/+/30442 Reviewed-by: Yusuf Mohsinally <mohsinally@google.com> Commit-Queue: Yusuf Mohsinally <mohsinally@google.com>
Contains configuration and tools related to Flutter's build infrastructure.
Configuration is generated by lucicfg, which is part of depot_tools. Rather than updating the individual .cfg files, updates should be made to config/main.star. That file can then be run to regenerate the configuration.
.star files (do not edit .cfg files; those are generated)../config/main.star..cfg files have been updated (e.g. via git status)..star files and .cfg files.This section talks about test shards defined in dev/bots/test.dart used to split framework tests into smaller chunks that can be tested in parallel. This does not refer to LUCI/Swarming shards.
When you add a new shard it does not automatically run in LUCI. To add it to LUCI you need to add a new builder in the framework_config.star. Typically, you'll add two builders, one “try” builder that tests PRs, and one “prod” builder that tests submitted PRs on the master branch.
Here's an example of a “try” builder:
common.linux_try_builder(
name = "Linux web_tests|web_tests",
recipe = "flutter/flutter",
repo = repos.FLUTTER,
list_view_name = list_view_name,
properties = {
"shard": "web_tests",
"subshards": ["0", "1", "2", "3", "4", "5", "6", "7_last"],
"dependencies": [{"dependency": "android_sdk"}, {"dependency": "chrome_and_driver"}, {"dependency": "goldctl"}],
},
caches = [
swarming.cache(name = "pub_cache", path = ".pub_cache"),
swarming.cache(name = "android_sdk", path = "android29"),
],
)
The easiest way to add a new one is to copy an existing builder and tweak the parameters.
As of today, it is not possible to test configuration changes until after they propagated to LUCI. One way to mitigate the risk of breakage is to run your test shard using an existing builder. To do that, find a builder in one of the .star files (e.g. framework_config.star) that satisfies the dependencies of the new test shard, then use the led tool (bundled with depot_tools) to run it.
For example, let‘s say you are adding an integration test shard for web, and the new shard needs a Linux builder with Chrome and ChromeDriver in order to run. Luckily, there’s already the Linux web_tests builder that satisfies these dependencies. To test the new shard, use the Linux web_tests builder, but ask it to run a different shards and/or subshard. Let‘s say the new test shard is called “foo”, it has 3 subshards (0, 1, 2_last) and it’s currently staged in pull request 99999. You can run it using the following led command:
led get-builder "luci.flutter.try:Linux web_tests" \ | led edit -pa git_ref='refs/pull/99999/head' \ | led edit -pa git_url='https://github.com/flutter/flutter' \ | led edit -pa shard='foo' \ | led edit -pa subshard='1' \ | led launch
One may speed up the propagation of the new configuration files by visiting the luci-config web ui, logging in, and searching for projects/flutter. from there, one may click on the projects/flutter search result and click the download icon to force an update.