Skip to content
This repository was archived by the owner on Feb 25, 2025. It is now read-only.

Remove Mac mac_android_aot_engine in favor of Linux #41210

Merged
merged 1 commit into from
Apr 14, 2023

Conversation

jmagman
Copy link
Member

@jmagman jmagman commented Apr 14, 2023

Linux mac_android_aot_engine is passing: https://ci.chromium.org/p/flutter/builders/try/Linux%20mac_android_aot_engine/1

Remove bringup and remove Mac mac_android_aot_engine in favor of the Linux orchestrator #41181

@jmagman jmagman self-assigned this Apr 14, 2023
@jmagman jmagman marked this pull request as ready for review April 14, 2023 18:52
@jmagman jmagman requested review from zanderso and keyonghan April 14, 2023 18:52
Copy link
Contributor

@keyonghan keyonghan left a comment

Choose a reason for hiding this comment

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

LGTM

@jmagman jmagman added the autosubmit Merge PR when tree becomes green via auto submit App label Apr 14, 2023
@auto-submit auto-submit bot merged commit 4a3433f into flutter:main Apr 14, 2023
@jmagman jmagman deleted the remove-mac-android_aot_engine branch April 14, 2023 19:08
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Apr 14, 2023
auto-submit bot pushed a commit that referenced this pull request Apr 14, 2023
`mac_unopt` orchestrator kicks off Mac swarming tasks, but don't seem to need to run on a Mac.  Swap Linux bots which are more plentiful and have a shorter queue time.

Start by adding new `bringup` builder to see if it passes in staging.  If it passes I will remove the Mac variant ASAP.

Led run: https://luci-milo.appspot.com/raw/build/logs.chromium.org/flutter/led/keyonghan_google.com/49f579a1636ddec2ffc05cb6053196a4472f5e4f69034386f88824356c479621/+/build.proto

Keep `mac_ios_engine` and `mac_host_engine` as Macs since they need to run Xcodes on Macs to create the xcframeworks.

See also #41181 and #41210
auto-submit bot pushed a commit to flutter/flutter that referenced this pull request Apr 14, 2023
…124896)

flutter/engine@413e95d...e9739bc

2023-04-14 [email protected] [macOS] Build unittests on all macOS host builds (flutter/engine#41215)
2023-04-14 [email protected] [web] Don't run goldctl init more than once (flutter/engine#41207)
2023-04-14 [email protected] Roll Skia from 2bd12e31d578 to 22e417bea884 (4 revisions) (flutter/engine#41213)
2023-04-14 [email protected] Revert "[Android] Send connectionClosed message when keyboard becomes invisible to ensure framework focus state is correct." (flutter/engine#41211)
2023-04-14 [email protected] Remove `Mac mac_android_aot_engine` in favor of Linux (flutter/engine#41210)
2023-04-14 [email protected] Run clang tidy builds on arm Macs (flutter/engine#41183)
2023-04-14 [email protected] Roll Fuchsia Linux SDK from diD1gLr_dKWFJlsSn... to Z0of2S9pf3Zn1nsJP... (flutter/engine#41209)

Also rolling transitive DEPS:
  fuchsia/sdk/core/linux-amd64 from diD1gLr_dKWF to Z0of2S9pf3Zn

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

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

To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
exaby73 pushed a commit to exaby73/flutter_nevercode that referenced this pull request Apr 17, 2023
…lutter#124896)

flutter/engine@413e95d...e9739bc

2023-04-14 [email protected] [macOS] Build unittests on all macOS host builds (flutter/engine#41215)
2023-04-14 [email protected] [web] Don't run goldctl init more than once (flutter/engine#41207)
2023-04-14 [email protected] Roll Skia from 2bd12e31d578 to 22e417bea884 (4 revisions) (flutter/engine#41213)
2023-04-14 [email protected] Revert "[Android] Send connectionClosed message when keyboard becomes invisible to ensure framework focus state is correct." (flutter/engine#41211)
2023-04-14 [email protected] Remove `Mac mac_android_aot_engine` in favor of Linux (flutter/engine#41210)
2023-04-14 [email protected] Run clang tidy builds on arm Macs (flutter/engine#41183)
2023-04-14 [email protected] Roll Fuchsia Linux SDK from diD1gLr_dKWFJlsSn... to Z0of2S9pf3Zn1nsJP... (flutter/engine#41209)

Also rolling transitive DEPS:
  fuchsia/sdk/core/linux-amd64 from diD1gLr_dKWF to Z0of2S9pf3Zn

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

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

To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
itsjustkevin pushed a commit to itsjustkevin/engine that referenced this pull request Apr 25, 2023
`Linux mac_android_aot_engine` is passing: https://ci.chromium.org/p/flutter/builders/try/Linux%20mac_android_aot_engine/1

Remove `bringup` and remove `Mac mac_android_aot_engine` in favor of the Linux orchestrator flutter#41181
itsjustkevin added a commit that referenced this pull request Apr 26, 2023
…1489)

# Flutter beta 3.10.0-1.4.pre Engine

## Scheduled Cherrypicks

- Roll dart revision: dart-lang/sdk@d2b8bf9c2
- #41210 
- Updates SKIA revision to `0d31aa1f49cb8cfd790c44d92a9a023839524865`

---------

Co-authored-by: Jenn Magder <[email protected]>
drewroengoogle pushed a commit that referenced this pull request Jul 25, 2023
`mac_unopt` orchestrator kicks off Mac swarming tasks, but don't seem to need to run on a Mac.  Swap Linux bots which are more plentiful and have a shorter queue time.

Start by adding new `bringup` builder to see if it passes in staging.  If it passes I will remove the Mac variant ASAP.

Led run: https://luci-milo.appspot.com/raw/build/logs.chromium.org/flutter/led/keyonghan_google.com/49f579a1636ddec2ffc05cb6053196a4472f5e4f69034386f88824356c479621/+/build.proto

Keep `mac_ios_engine` and `mac_host_engine` as Macs since they need to run Xcodes on Macs to create the xcframeworks.

See also #41181 and #41210
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
autosubmit Merge PR when tree becomes green via auto submit App
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants