Skip to content

[ci] Increase test timeout for Mac_x64 build_tests shards#188804

Merged
auto-submit[bot] merged 2 commits into
flutter:masterfrom
bkonyi:increase-test-timeout-188242
Jul 1, 2026
Merged

[ci] Increase test timeout for Mac_x64 build_tests shards#188804
auto-submit[bot] merged 2 commits into
flutter:masterfrom
bkonyi:increase-test-timeout-188242

Conversation

@bkonyi

@bkonyi bkonyi commented Jun 30, 2026

Copy link
Copy Markdown
Contributor

Increase test_timeout_secs to 2700 (45 minutes) for all five Mac_x64 build_tests shards (1_5 to 5_5) in .ci.yaml.

The default test timeout is 30 minutes. Shard 2_5 has been taking close to 29 minutes, leading to timeouts. Since test sharding distribution shuffles when tests are added/removed, all five shards have been updated.

Setting test_timeout_secs to 2700 provides a 15-minute buffer before the global LUCI task timeout of 60 minutes, ensuring the test runner itself times out and prints diagnostics.

Fixes #188242

Increase `test_timeout_secs` to `2700` (45 minutes) for all five `Mac_x64 build_tests` shards (`1_5` to `5_5`) in `.ci.yaml`.

The default test timeout is 30 minutes. Shard `2_5` has been taking close to 29 minutes, leading to timeouts. Since test sharding distribution shuffles when tests are added/removed, all five shards have been updated.

Setting `test_timeout_secs` to `2700` provides a 15-minute buffer before the global LUCI task timeout of 60 minutes, ensuring the test runner itself times out and prints diagnostics.

Fixes flutter#188242
@flutter-dashboard flutter-dashboard Bot added the CICD Run CI/CD label Jun 30, 2026
@bkonyi bkonyi requested a review from jtmcdole June 30, 2026 17:51

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Code Review

This pull request updates the .ci.yaml configuration file to add a test_timeout_secs property with a value of "2700" to five target configurations. There are no review comments, and I have no feedback to provide.

@reidbaker

Copy link
Copy Markdown
Contributor

this pr will need to be cherry picked into stable to release 3.44.5

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

autosubmit Merge PR when tree becomes green via auto submit App CICD Run CI/CD cp: stable cherry pick this pull request to stable release candidate branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Consider increasing the test timeout for Mac_x64 build_tests_2_5 or resharding

2 participants