Skip to content

Conversation

@jtmcdole
Copy link
Member

Mac bots in device lab were upgraded to 15.7; which no longer matches ci.yaml

question for future: is there a better option for this other than pinning?

fixes: #178384

Mac bots in device lab were upgraded to 15.7; which no longer matches
ci.yaml

question for future: is there a better option for this other than
pinning?

fixes: flutter#178384
@jtmcdole jtmcdole added the emergency Jump the queue; land PR in front of all others; only use for emergencies label Nov 12, 2025
@flutter-dashboard
Copy link

Detected the emergency label.

If you add the autosubmit label, the bot will wait until all presubmits pass but ignore the tree status, allowing fixes for tree breakages while still validating that they don't break any existing presubmits.

The "Merge" button is also unlocked. To bypass presubmits as well as the tree status, press the GitHub "Add to Merge Queue".

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 updates the CI configuration in .ci.yaml to add Mac-15.7 to the list of supported operating systems. This change directly addresses an issue where CI jobs were failing due to a mismatch with upgraded Mac bots. My review includes a suggestion to improve the long-term maintainability of this configuration by investigating more flexible OS version matching to prevent similar issues in the future.

{"dependency": "open_jdk", "version": "version:21"}
]
os: Mac-15.1|Mac-15.5|Mac-15.6
os: Mac-15.1|Mac-15.5|Mac-15.6|Mac-15.7
Copy link
Contributor

Choose a reason for hiding this comment

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

medium

While this fixes the immediate problem, pinning specific minor OS versions is brittle and requires manual updates for each new version, as you noted in the description. To improve maintainability, it would be ideal to use a more flexible pattern if the CI system supports it. For instance, a wildcard like Mac-15.* or specifying only the major version could make this configuration more resilient to future OS upgrades on the bots.

@jtmcdole jtmcdole added this pull request to the merge queue Nov 12, 2025
Merged via the queue into flutter:master with commit 5884de7 Nov 12, 2025
151 of 156 checks passed
@jtmcdole jtmcdole deleted the fixTreeOutage branch November 12, 2025 17:28
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Nov 12, 2025
@jtmcdole jtmcdole added the cp: beta cherry pick this pull request to beta release candidate branch label Nov 12, 2025
flutteractionsbot pushed a commit to flutteractionsbot/flutter that referenced this pull request Nov 12, 2025
Mac bots in device lab were upgraded to 15.7; which no longer matches
ci.yaml

question for future: is there a better option for this other than
pinning?

fixes: flutter#178384
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Nov 12, 2025
jtmcdole added a commit to jtmcdole/flutter that referenced this pull request Nov 12, 2025
Mac bots in device lab were upgraded to 15.7; which no longer matches
ci.yaml

question for future: is there a better option for this other than
pinning?

fixes: flutter#178384
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Nov 13, 2025
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Nov 13, 2025
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Nov 13, 2025
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Nov 13, 2025
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Nov 14, 2025
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Nov 14, 2025
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Nov 14, 2025
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Nov 14, 2025
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Nov 15, 2025
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Nov 15, 2025
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Nov 16, 2025
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Nov 16, 2025
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Nov 16, 2025
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Nov 17, 2025
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Nov 17, 2025
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Nov 17, 2025
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Nov 17, 2025
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Nov 17, 2025
auto-submit bot pushed a commit that referenced this pull request Nov 17, 2025
Cherry picks #178387 resolving #178384 - mac bots upgraded to 15.7
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Nov 17, 2025
IvoneDjaja pushed a commit to IvoneDjaja/flutter that referenced this pull request Nov 22, 2025
Mac bots in device lab were upgraded to 15.7; which no longer matches
ci.yaml

question for future: is there a better option for this other than
pinning?

fixes: flutter#178384
mboetger pushed a commit to mboetger/flutter that referenced this pull request Dec 2, 2025
Mac bots in device lab were upgraded to 15.7; which no longer matches
ci.yaml

question for future: is there a better option for this other than
pinning?

fixes: flutter#178384
reidbaker pushed a commit to AbdeMohlbi/flutter that referenced this pull request Dec 10, 2025
Mac bots in device lab were upgraded to 15.7; which no longer matches
ci.yaml

question for future: is there a better option for this other than
pinning?

fixes: flutter#178384
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cp: beta cherry pick this pull request to beta release candidate branch emergency Jump the queue; land PR in front of all others; only use for emergencies

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[infra] Out of resources to run certain tests

3 participants