-
Notifications
You must be signed in to change notification settings - Fork 29.8k
Marks Mac_arm64_mokey run_release_test to be flaky #177372
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.
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
Conversation
There was a problem hiding this 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 marks the Mac_arm6g4_mokey run_release_test as flaky by setting bringup: true. This is a standard and correct approach to handle flaky tests in the CI. My review includes a minor suggestion to align the comment with the repository's documented process for tracking flaky tests, which will improve maintainability.
|
|
||
| # mac mokey test | ||
| - name: Mac_arm64_mokey run_release_test | ||
| bringup: true # Flaky https://github.com/flutter/flutter/issues/177371 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
To improve tracking and maintainability of this flaky test, the comment should be on its own line above bringup: true and follow the TODO(username): issue url format as specified in the repository's guide on reducing test flakiness.1
# TODO(flutter-hackers): Flaky https://github.com/flutter/flutter/issues/177371
bringup: trueStyle Guide References
Footnotes
-
The documentation on reducing test flakiness specifies that when a test is marked as flaky, a
TODOcomment with the username and issue URL should be added above thebringup: trueline to ensure proper tracking. ↩
Piinks
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
<!-- meta-tags: To be used by the automation script only, DO NOT MODIFY.
{
"name": "Mac_arm64_mokey run_release_test"
}
-->
Issue link: flutter#177371
Manual roll requested by tarrinneal@google.com flutter/flutter@b2de367...6a1f5b7 2025-12-09 31510811+jwlilly@users.noreply.github.com Fix - Semantics focus does not move outside viewport when moving up or down (flutter/flutter#179480) 2025-12-09 ahmedsameha1@gmail.com Make sure that a CupertinoActionSheetAction doesn't crash in 0x0 envi… (flutter/flutter#178955) 2025-12-09 ahmedsameha1@gmail.com Make sure that a CupertinoPickerDefaultSelectionOverlay doesn't crash… (flutter/flutter#179351) 2025-12-09 ahmedsameha1@gmail.com Make sure that a CupertinoExpansionTile doesn't crash in 0x0 environment (flutter/flutter#178978) 2025-12-09 engine-flutter-autoroll@skia.org Roll Packages from 33a9a81 to 338ecd3 (5 revisions) (flutter/flutter#179625) 2025-12-09 kjlubick@users.noreply.github.com [skia] Update SkSerialProcs to use new type (flutter/flutter#179347) 2025-12-09 engine-flutter-autoroll@skia.org Roll Skia from 895fa7417947 to 502ee6f2a0d7 (4 revisions) (flutter/flutter#179617) 2025-12-09 engine-flutter-autoroll@skia.org Roll Skia from 9f276dfa0bdc to 895fa7417947 (1 revision) (flutter/flutter#179608) 2025-12-09 1961493+harryterkelsen@users.noreply.github.com Remove unused optional argument in _followDiagnosticableChain (flutter/flutter#179525) 2025-12-08 engine-flutter-autoroll@skia.org Roll Dart SDK from 3c07646cdcb9 to 019cb923bf62 (1 revision) (flutter/flutter#179595) 2025-12-08 fluttergithubbot@gmail.com Marks Linux_mokey flutter_engine_group_performance to be flaky (flutter/flutter#179115) 2025-12-08 engine-flutter-autoroll@skia.org Roll Skia from 00e6fc407968 to 9f276dfa0bdc (3 revisions) (flutter/flutter#179594) 2025-12-08 fluttergithubbot@gmail.com Marks Mac_arm64_mokey run_release_test to be flaky (flutter/flutter#177372) 2025-12-08 engine-flutter-autoroll@skia.org Roll Skia from b1936c760645 to 00e6fc407968 (5 revisions) (flutter/flutter#179589) 2025-12-08 kevmoo@users.noreply.github.com MatrixUtils.forceToPoint - simplify and optimize (flutter/flutter#179546) 2025-12-08 97480502+b-luk@users.noreply.github.com Change GenerateFilledArcStrip to use non-overlapping triangles (flutter/flutter#179292) 2025-12-08 matt.boetger@gmail.com Android implementation of content sizing (flutter/flutter#176063) 2025-12-08 engine-flutter-autoroll@skia.org Roll Dart SDK from 75899721aa42 to 3c07646cdcb9 (1 revision) (flutter/flutter#179587) 2025-12-08 matt.boetger@gmail.com Redistribute TESTOWNERS for Android team (flutter/flutter#179464) 2025-12-08 ahmedsameha1@gmail.com Make sure that a CupertinoListTile doesn't crash in 0x0 environment (flutter/flutter#179109) 2025-12-08 ahmedsameha1@gmail.com Make sure that a CupertinoFocusHalo doesn't crash in 0x0 environment (flutter/flutter#178773) 2025-12-08 ahmedsameha1@gmail.com Make sure that a CupertinoPopupSurface doesn't crash in 0x0 environment (flutter/flutter#178929) 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 stuartmorgan@google.com,tarrinneal@google.com 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
Issue link: #177371