Skip to content

consolidate all web integration tests under the same shard - #82307

Merged
yjbanov merged 6 commits into
flutter:masterfrom
yjbanov:consolidate-web-integration-tests
May 12, 2021
Merged

yjbanov merged 6 commits into
flutter:masterfrom
yjbanov:consolidate-web-integration-tests

Conversation

@yjbanov

@yjbanov yjbanov commented May 12, 2021

Copy link
Copy Markdown
Contributor
  • Increase the number of sub-shards in web_long_running_tests from 3 to 5 and merge all remaining integration tests under it.
  • Remove the web_integration_tests shard (it doesn't scale well because it doesn't support sub-shards)

@yjbanov
yjbanov requested review from godofredoc and keyonghan May 12, 2021 00:34
@flutter-dashboard flutter-dashboard Bot added the c: contributor-productivity Team-specific productivity, code health, technical debt. label May 12, 2021
@google-cla google-cla Bot added the cla: yes label May 12, 2021

@keyonghan keyonghan left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@yjbanov
yjbanov force-pushed the consolidate-web-integration-tests branch 3 times, most recently from 64a0c81 to 3ceb3f6 Compare May 12, 2021 17:35
@yjbanov
yjbanov force-pushed the consolidate-web-integration-tests branch from 3ceb3f6 to b91dcc3 Compare May 12, 2021 17:52
@fluttergithubbot

Copy link
Copy Markdown
Contributor

This pull request is not suitable for automatic merging in its current state.

  • The status or check suite Linux web_long_running_tests_1_5 has failed. Please fix the issues identified (or deflake) before re-applying this label.
  • The status or check suite Linux analyze has failed. Please fix the issues identified (or deflake) before re-applying this label.
  • The status or check suite analyze-linux has failed. Please fix the issues identified (or deflake) before re-applying this label.

@yjbanov

yjbanov commented May 12, 2021

Copy link
Copy Markdown
Contributor Author

Landing now to unblock PRs. Pending stuff is not related to web.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

c: contributor-productivity Team-specific productivity, code health, technical debt.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants