Skip to content

Remove bringup flag from Mac_arm64_mokey task#178759

Merged
auto-submit[bot] merged 2 commits intomasterfrom
camsim99-patch-1
Jan 21, 2026
Merged

Remove bringup flag from Mac_arm64_mokey task#178759
auto-submit[bot] merged 2 commits intomasterfrom
camsim99-patch-1

Conversation

@camsim99
Copy link
Contributor

@camsim99 camsim99 commented Nov 18, 2025

Removed the 'bringup' flag from Mac_arm64_mokey task due to flakiness. The flakiness seems to be fixedb y #180645.

Fixes #177384.

Pre-launch Checklist

If you need help, consider asking for advice on the #hackers-new channel on Discord.

Note: The Flutter team is currently trialing the use of Gemini Code Assist for GitHub. Comments from the gemini-code-assist bot should not be taken as authoritative feedback from the Flutter team. If you find its comments useful you can update your code accordingly, but if you are unsure or disagree with the feedback, please feel free to wait for a Flutter team member's review for guidance on which automated comments should be addressed.

Removed the 'bringup' flag from Mac_arm64_mokey task due to flakiness.
@camsim99 camsim99 marked this pull request as ready for review January 21, 2026 18:16
@camsim99 camsim99 requested a review from a team January 21, 2026 18:16
Copy link
Contributor

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

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 the Mac_arm64_mokey run_debug_test_android CI target in .ci.yaml. This change promotes the test out of "bringup" mode, meaning its failures in post-submit will now be considered tree-breaking. This is done based on the assumption that the test is no longer flaky.

@gmackall
Copy link
Member

A lot of the flakes listed in #177384 (comment) are in December, while the fix for #178384 seems to have landed in November. Do we have reason to believe the fix did not take effect until recently?

@camsim99
Copy link
Contributor Author

A lot of the flakes listed in #177384 (comment) are in December, while the fix for #178384 seems to have landed in November. Do we have reason to believe the fix did not take effect until recently?

Oops sorry I didn't update the description! I think this was actually fixed by #180645 -- will update

Copy link
Member

@gmackall gmackall left a comment

Choose a reason for hiding this comment

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

nice, lgtm!

@camsim99 camsim99 added the autosubmit Merge PR when tree becomes green via auto submit App label Jan 21, 2026
@auto-submit auto-submit bot added this pull request to the merge queue Jan 21, 2026
Merged via the queue into master with commit 3f2548b Jan 21, 2026
157 checks passed
@auto-submit auto-submit bot deleted the camsim99-patch-1 branch January 21, 2026 22:35
@flutter-dashboard flutter-dashboard bot removed the autosubmit Merge PR when tree becomes green via auto submit App label Jan 21, 2026
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Jan 22, 2026
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Jan 22, 2026
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Jan 22, 2026
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Jan 23, 2026
flutter-zl pushed a commit to flutter-zl/flutter that referenced this pull request Feb 10, 2026
Removed the 'bringup' flag from Mac_arm64_mokey task due to flakiness.
The flakiness seems to be fixedb y
flutter#180645.

Fixes flutter#177384.

## Pre-launch Checklist

- [x] I read the [Contributor Guide] and followed the process outlined
there for submitting PRs.
- [x] I read the [Tree Hygiene] wiki page, which explains my
responsibilities.
- [x] I read and followed the [Flutter Style Guide], including [Features
we expect every widget to implement].
- [x] I signed the [CLA].
- [x] I listed at least one issue that this PR fixes in the description
above.
- [x] I updated/added relevant documentation (doc comments with `///`).
- [x] I added new tests to check the change I am making, or this PR is
[test-exempt].
- [x] I followed the [breaking change policy] and added [Data Driven
Fixes] where supported.
- [x] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel
on [Discord].

**Note**: The Flutter team is currently trialing the use of [Gemini Code
Assist for
GitHub](https://developers.google.com/gemini-code-assist/docs/review-github-code).
Comments from the `gemini-code-assist` bot should not be taken as
authoritative feedback from the Flutter team. If you find its comments
useful you can update your code accordingly, but if you are unsure or
disagree with the feedback, please feel free to wait for a Flutter team
member's review for guidance on which automated comments should be
addressed.

<!-- Links -->
[Contributor Guide]:
https://github.com/flutter/flutter/blob/main/docs/contributing/Tree-hygiene.md#overview
[Tree Hygiene]:
https://github.com/flutter/flutter/blob/main/docs/contributing/Tree-hygiene.md
[test-exempt]:
https://github.com/flutter/flutter/blob/main/docs/contributing/Tree-hygiene.md#tests
[Flutter Style Guide]:
https://github.com/flutter/flutter/blob/main/docs/contributing/Style-guide-for-Flutter-repo.md
[Features we expect every widget to implement]:
https://github.com/flutter/flutter/blob/main/docs/contributing/Style-guide-for-Flutter-repo.md#features-we-expect-every-widget-to-implement
[CLA]: https://cla.developers.google.com/
[flutter/tests]: https://github.com/flutter/tests
[breaking change policy]:
https://github.com/flutter/flutter/blob/main/docs/contributing/Tree-hygiene.md#handling-breaking-changes
[Discord]:
https://github.com/flutter/flutter/blob/main/docs/contributing/Chat.md
[Data Driven Fixes]:
https://github.com/flutter/flutter/blob/main/docs/contributing/Data-driven-Fixes.md
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Mac_arm64_mokey run_debug_test_android is 7.22% flaky

2 participants