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

Conversation

@dnfield
Copy link
Contributor

@dnfield dnfield commented Apr 13, 2023

fixes flutter/flutter#124678

those machines do not support compute subgroups. They make up only a relatively small fraction of the pool.

This does not restrict the arm machines, as they're already restricted by the cpu architecture dimension.

@dnfield dnfield requested review from godofredoc and zanderso April 13, 2023 15:13
Copy link
Member

@zanderso zanderso left a comment

Choose a reason for hiding this comment

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

I wish we could add comments to .json files to put a link to the issue explaining why the restriction is there!

@dnfield dnfield added the autosubmit Merge PR when tree becomes green via auto submit App label Apr 13, 2023
@auto-submit auto-submit bot removed the autosubmit Merge PR when tree becomes green via auto submit App label Apr 13, 2023
@auto-submit
Copy link
Contributor

auto-submit bot commented Apr 13, 2023

auto label is removed for flutter/engine, pr: 41159, due to - The status or check suite Linux linux_web_engine has failed. Please fix the issues identified (or deflake) before re-applying this label.

@keyonghan
Copy link
Contributor

Seems the change is not picked up by the pre-submit build. https://ci.chromium.org/ui/p/flutter/builders/try/Mac%20Engine%20Drone/160302/overview still uses the old drone_dimension. Do you need a bug? /cc @godofredoc

@dnfield
Copy link
Contributor Author

dnfield commented Apr 13, 2023

Seems the change is not picked up by the pre-submit build. https://ci.chromium.org/ui/p/flutter/builders/try/Mac%20Engine%20Drone/160302/overview still uses the old drone_dimension. Do you need a bug? /cc @godofredoc

That one is allowed to run on the old ones - I only updated the dimensions for the drones that run Host tests for host*

@dnfield dnfield added the autosubmit Merge PR when tree becomes green via auto submit App label Apr 13, 2023
@keyonghan
Copy link
Contributor

Seems the change is not picked up by the pre-submit build. https://ci.chromium.org/ui/p/flutter/builders/try/Mac%20Engine%20Drone/160302/overview still uses the old drone_dimension. Do you need a bug? /cc @godofredoc

I took this back. The change takes effect right away. Here is the task where the dimension is enforced: https://chromium-swarm.appspot.com/task?id=619400c517759a10&o=true&w=true

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

@dnfield dnfield merged commit f5dab1b into flutter:main Apr 13, 2023
@dnfield dnfield deleted the flake branch April 13, 2023 17:45
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Apr 13, 2023
dnfield added a commit that referenced this pull request Apr 14, 2023
This target also runs impeller_unittests that aren't safe to run right
now on Macmini7,1

I missed this in #41159
zhongwuzw pushed a commit to zhongwuzw/engine that referenced this pull request Apr 14, 2023
fixes flutter/flutter#124678

those machines do not support compute subgroups. They make up only a
relatively small fraction of the pool.

This does not restrict the arm machines, as they're already restricted
by the cpu architecture dimension.
zhongwuzw pushed a commit to zhongwuzw/engine that referenced this pull request Apr 14, 2023
This target also runs impeller_unittests that aren't safe to run right
now on Macmini7,1

I missed this in flutter#41159
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.

Compute/ComputeSubgroupTest.* from impeller_unittests cannot run on Iris 5100 mac minis

3 participants