Skip to content

[ci] Bring up LUCI Linux custom package tests #4382

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 9 commits into from
Jul 6, 2023
Merged

[ci] Bring up LUCI Linux custom package tests #4382

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
9 commits
Select commit Hold shift + click to select a range
576f787
Remove custom tests that now duplicate standard tests
stuartmorgan-g Jul 5, 2023
edd434c
Clean up exclusions, which are no longer needed
stuartmorgan-g Jul 5, 2023
474a231
Add script
stuartmorgan-g Jul 5, 2023
b9b0fb7
Add new LUCI targets
stuartmorgan-g Jul 5, 2023
7981f40
DO NOT LAND - test in existing target
stuartmorgan-g Jul 5, 2023
f19a528
Add deps to the experiment
stuartmorgan-g Jul 5, 2023
8e123fe
copypasta in added dependencies
stuartmorgan-g Jul 5, 2023
4f072bb
Revert presubmit test changes, update new packages
stuartmorgan-g Jul 5, 2023
c3d4f01
Typo
stuartmorgan-g Jul 6, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
34 changes: 34 additions & 0 deletions .ci.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -214,6 +214,40 @@ targets:
channel: stable
version_file: flutter_stable.version

- name: Linux_android custom_package_tests master
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Does the name of these need to start with Linux_android? Why not just Linux custom_package_tests master?

Copy link
Contributor Author

@stuartmorgan-g stuartmorgan-g Jul 6, 2023 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That would use the more generic Linux configuration, and then I would have to duplicate all the dependencies in each of the targets. I considered it, but opted for less repetition over preferred naming.

The rigidity of LUCI .ci.yaml naming and how it determines behavior is definitely not one of my favorite parts of LUCI. I would much rather the base config be a property unrelated to name.

bringup: true # New target
recipe: packages/packages
timeout: 30
properties:
add_recipes_cq: "true"
version_file: flutter_master.version
target_file: linux_custom_package_tests.yaml
cores: "32"
# Pigeon tests need Andoid deps (thus the Linux_android base) and
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
# Pigeon tests need Andoid deps (thus the Linux_android base) and
# Pigeon tests need Android deps (thus the Linux_android base) and

Is this sufficient to name the suite "Linux_android" though? :P

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I like andoid better

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Linux_andoid custom_package_tests master :P

# clang-format.
dependencies: >-
[
{"dependency": "clang", "version": "git_revision:5d5aba78dbbee75508f01bcaa69aedb2ab79065a"}
]
channel: master

- name: Linux_android custom_package_tests stable
bringup: true # New target
recipe: packages/packages
timeout: 30
properties:
add_recipes_cq: "true"
version_file: flutter_stable.version
target_file: linux_custom_package_tests.yaml
cores: "32"
# Pigeon tests need Android deps (thus the Linux_android base) and
# clang-format.
dependencies: >-
[
{"dependency": "clang", "version": "git_revision:5d5aba78dbbee75508f01bcaa69aedb2ab79065a"}
]
channel: stable

### Android tasks ###
- name: Linux_android android_build_all_packages master
recipe: packages/packages
Expand Down
8 changes: 1 addition & 7 deletions .ci/scripts/custom_package_tests.sh
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4,11 +4,5 @@
# found in the LICENSE file.
set -e

# Exclusions
#
# script/configs/linux_only_custom_test.yaml
# Custom tests need Chrome. (They run in linux-custom_package_tests)

dart ./script/tool/bin/flutter_plugin_tools.dart custom-test \
--packages-for-branch --log-timing \
--exclude=script/configs/linux_only_custom_test.yaml
--packages-for-branch --log-timing
6 changes: 6 additions & 0 deletions .ci/targets/linux_custom_package_tests.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,6 @@
tasks:
- name: prepare tool
script: .ci/scripts/prepare_tool.sh
- name: custom tests
script: script/tool_runner.sh
args: ["custom-test"]
50 changes: 0 additions & 50 deletions packages/cross_file/tool/run_tests.dart
View file
Open in desktop

This file was deleted.

50 changes: 0 additions & 50 deletions packages/google_identity_services_web/tool/run_tests.dart
View file
Open in desktop

This file was deleted.

9 changes: 0 additions & 9 deletions script/configs/linux_only_custom_test.yaml
View file
Open in desktop

This file was deleted.