Skip to content

[ci] mac build_test bringup false#184738

Merged
auto-submit[bot] merged 1 commit into
flutter:masterfrom
dcharkes:mac-build-test-bringup-false
Apr 8, 2026
Merged

[ci] mac build_test bringup false#184738
auto-submit[bot] merged 1 commit into
flutter:masterfrom
dcharkes:mac-build-test-bringup-false

Conversation

@dcharkes

@dcharkes dcharkes commented Apr 7, 2026

Copy link
Copy Markdown
Contributor

We resharded the mac build-tests in #184719.

That required bringup: true, the bots are green for the last 3 commits, move them out of bringup.

image

The timeouts seem better this time. The slowest shard runs in 31 minutes.

@dcharkes dcharkes added the CICD Run CI/CD label Apr 7, 2026
@dcharkes dcharkes added the autosubmit Merge PR when tree becomes green via auto submit App label Apr 7, 2026

@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 removes the bringup: true flag from several Mac x64 and arm64 build test shards in .ci.yaml, making these targets blocking. Feedback highlights that Mac_arm64 build_tests_1_5 is inconsistent with other shards because it lacks the add_recipes_cq property and remains disabled in presubmit, which may result in a gap in test coverage for that shard.

Comment thread .ci.yaml

- name: Mac_arm64 build_tests_1_5
recipe: flutter/flutter_drone
bringup: true

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.

medium

The target Mac_arm64 build_tests_1_5 is being moved out of bringup (making it blocking), but it remains disabled in presubmit (presubmit: false) with a comment citing a high flake rate. Since the Mac_x64 shards also skip presubmit (relying on ARM), this results in Shard 1 having no presubmit coverage at all. Additionally, this shard is missing add_recipes_cq: "true", which is present in shards 2-5. If the recent resharding has stabilized these tests, consider re-enabling presubmit, adding the CQ property, and removing the flake-related comments to ensure consistent coverage. If it remains flaky, it may be premature to make it blocking.

@auto-submit auto-submit Bot added this pull request to the merge queue Apr 8, 2026
Merged via the queue into flutter:master with commit f50c79b Apr 8, 2026
152 of 153 checks passed
@flutter-dashboard flutter-dashboard Bot removed the autosubmit Merge PR when tree becomes green via auto submit App label Apr 8, 2026
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Apr 8, 2026
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Apr 8, 2026
github-merge-queue Bot pushed a commit that referenced this pull request Apr 8, 2026
#150642 has been closed saying
that it's fixed.

And also I resharded these bots
(#184719 &
#184738), which likely means
shard 1 now runs different tests than before.

FYI @vashworth @gmackall
auto-submit Bot pushed a commit to flutter/packages that referenced this pull request Apr 8, 2026
flutter/flutter@a0924c7...05e0ae0

2026-04-08 [email protected] Fix Android engine flags defaulting to true for malformed values (flutter/flutter#184631)
2026-04-08 [email protected] Try one more again (flutter/flutter#184767)
2026-04-08 [email protected] Remove custom `analysis_options.yaml` from `imitation_game_flutter` (flutter/flutter#184717)
2026-04-08 [email protected] Add more error handling to unawaited callsites (flutter/flutter#184526)
2026-04-08 [email protected] Refactor: remove material from absorb_ponter_test, container_test,  lookup_boundary_test, page_view_test, router_test, semantics_clipping_test, semantics_merge_test, shadow_test, text_test (flutter/flutter#183309)
2026-04-08 [email protected] Remove editable_text_utils cross-imports from material and cupertino … (flutter/flutter#184519)
2026-04-08 [email protected] Replace hard coded max path length with system defined one. (flutter/flutter#184697)
2026-04-08 [email protected] [Re-land] Add Support For Built-in Kotlin (flutter/flutter#184745)
2026-04-08 [email protected] Manually stop and continue LLDB breakpoints on Xcode 26.4+ (flutter/flutter#184690)
2026-04-08 [email protected] Code freeze workflow (flutter/flutter#184246)
2026-04-08 [email protected] [Dot shorthands] Migrate examples/api/lib/widgets (flutter/flutter#183965)
2026-04-08 [email protected] [cupertino.dart] Implement CupertinoMenuAnchor and CupertinoMenuItem using RawMenuAnchor (flutter/flutter#182036)
2026-04-08 [email protected] [Semantics] clarify Android header docs (flutter/flutter#183573)
2026-04-08 [email protected] [ci] mac build_test bringup false (flutter/flutter#184738)
2026-04-08 [email protected] Reland "Apply rect clipping to surface views" (flutter/flutter#184732)
2026-04-08 [email protected] Remove bringup label for resharded Windows tool_integration_tests shards (flutter/flutter#184721)
2026-04-08 [email protected] Tool: Add search and filtering to widget preview scaffold (flutter/flutter#184023)
2026-04-08 [email protected] Update localization from translation console (flutter/flutter#184742)
2026-04-07 [email protected] Revert "Add Support For Built-in Kotlin (#184227)" (flutter/flutter#184739)
2026-04-07 [email protected] Collect HCPP adoption analytics for flutter run/build apk/build appbundle (flutter/flutter#184225)
2026-04-07 [email protected] Add a github workflow for reverting PRs. (flutter/flutter#184593)
2026-04-07 [email protected] Add Support For Built-in Kotlin (flutter/flutter#184227)
2026-04-07 [email protected] Revert "Apply rect clipping to surface views (#184471)" (flutter/flutter#184728)
2026-04-07 [email protected] [Fix-forward] Added Compose plugin to Add-to-app Integration Test (flutter/flutter#184681)

If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-packages
Please CC [email protected],[email protected] on the revert to ensure that a human
is aware of the problem.

To file a bug in Packages: https://github.com/flutter/flutter/issues/new/choose

To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
mbcorona pushed a commit to mbcorona/flutter that referenced this pull request Apr 15, 2026
We resharded the mac build-tests in
flutter#184719.

That required `bringup: true`, the bots are green for the last 3
commits, move them out of bringup.

<img width="362" height="148" alt="image"
src="https://github.com/user-attachments/assets/5f9b437c-3d93-42a4-960c-0f584fbd1adf"
/>

The timeouts seem better this time. The slowest shard runs in 31
minutes.
mbcorona pushed a commit to mbcorona/flutter that referenced this pull request Apr 15, 2026
flutter#150642 has been closed saying
that it's fixed.

And also I resharded these bots
(flutter#184719 &
flutter#184738), which likely means
shard 1 now runs different tests than before.

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

Labels

CICD Run CI/CD

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants