Skip to content

Commit cd44453

Browse files
authored
Be less verbose in CI for customer_testing, and revert old timeout hack (#139611)
This PR does two things. First, it reduces the verbosity in the customer_testing shard. We want to be less verbose because it's nigh on impossible to find anything when there's a failure in the verbose logs. Also, some additional comments are added to make following the breadcrumbs easier for the next person. Second, it reverts a timeout that had been set to 90 minutes a while ago. It's no longer needed. Fixes flutter/flutter#120901.
1 parent 9ed650a commit cd44453

File tree

2 files changed

+11
-2
lines changed

2 files changed

+11
-2
lines changed

.ci.yaml

Lines changed: 10 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,12 @@
33
# Flutter infra uses this file to generate a checklist of tasks to be performed
44
# for every commit.
55
#
6+
# The recipes mentioned below refer to those in this repo:
7+
# https://flutter.googlesource.com/recipes/+/refs/heads/main/recipes/
8+
#
9+
# The "flutter_drone" recipe just defers to dev/bots/test.dart in this repo,
10+
# with the shard set according to the "shard" key in this file.
11+
#
612
# More information at:
713
# * https://github.com/flutter/cocoon/blob/main/CI_YAML.md
814
enabled_branches:
@@ -420,6 +426,9 @@ targets:
420426
- .ci.yaml
421427

422428
- name: Linux customer_testing
429+
# This really just runs dev/bots/customer_testing/ci.sh,
430+
# but it does so indirectly via the flutter_drone recipe
431+
# calling the dev/bots/test.dart script.
423432
enabled_branches:
424433
- master
425434
recipe: flutter/flutter_drone
@@ -432,7 +441,7 @@ targets:
432441
- name: Linux docs_publish
433442
recipe: flutter/docs
434443
presubmit: false
435-
timeout: 90 # https://github.com/flutter/flutter/issues/120901
444+
timeout: 60
436445
dimensions:
437446
os: "Linux"
438447
properties:

dev/customer_testing/ci.sh

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -34,4 +34,4 @@ git clone https://github.com/flutter/tests.git ../../bin/cache/pkg/tests
3434
git -C ../../bin/cache/pkg/tests checkout `dart --enable-asserts ../tools/bin/find_commit.dart . master ../../bin/cache/pkg/tests main`
3535

3636
# Finally, run the tests.
37-
dart --enable-asserts run_tests.dart --verbose --skip-on-fetch-failure --skip-template ../../bin/cache/pkg/tests/registry/*.test
37+
dart --enable-asserts run_tests.dart --skip-on-fetch-failure --skip-template ../../bin/cache/pkg/tests/registry/*.test

0 commit comments

Comments
 (0)