Skip to content

Conversation

@dnfield
Copy link
Contributor

@dnfield dnfield commented Aug 8, 2019

We need to reduce the workload that the macs are doing at least temporarily.

/cc @tvolkert

@tvolkert
Copy link
Contributor

tvolkert commented Aug 8, 2019

So this:

  • Makes the MacOS integration tests only run on master (not pre-commit)
  • Skips running the widget tests on MacOS entirely -- presumably because they provide less value when compared to other tests we run on Macs?

What's the story with the removal of the "*_cache" lines?

@dnfield
Copy link
Contributor Author

dnfield commented Aug 8, 2019

We run the widget tests on LUCI on post submit.

The caches turned out to be frequently taking longer to create/upload/download than if we just download directly AFAICT, at least on mac. On Linux and Windows they seem better.

@dnfield
Copy link
Contributor Author

dnfield commented Aug 8, 2019

Redness is not real - going ot land this to kick the bots. Should also help us be less red for flakes.

@dnfield dnfield merged commit 9ae7625 into flutter:master Aug 8, 2019
@dnfield dnfield deleted the reduce_mac branch August 8, 2019 19:43
@jmagman jmagman mentioned this pull request Aug 9, 2019
9 tasks
@Hixie
Copy link
Contributor

Hixie commented Oct 3, 2019

Filed #41940 to track this.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 4, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants