Skip to content

Conversation

jiahaog
Copy link
Member

@jiahaog jiahaog commented Mar 29, 2021

Device lab tests for #76200

Tested:

First patch #76200, and cd dev/devicelab.

Android:

 ../../bin/cache/dart-sdk/bin/dart bin/run.dart -t bin/tasks/integration_test_test.dart

iOS

Need to modify the runner to create a custom team and bundle ID first, then

 ../../bin/cache/dart-sdk/bin/dart bin/run.dart -t bin/tasks/integration_test_test_ios.dart

Not sure if there's a way to test that the prod_builders.json is correct locally, so I think the above should be good enough

@flutter-dashboard flutter-dashboard bot added the c: contributor-productivity Team-specific productivity, code health, technical debt. label Mar 29, 2021
@google-cla google-cla bot added the cla: yes label Mar 29, 2021
@jiahaog jiahaog marked this pull request as ready for review March 30, 2021 07:28
@jiahaog jiahaog requested a review from jonahwilliams March 30, 2021 07:28
Copy link
Contributor

@jonahwilliams jonahwilliams left a comment

Choose a reason for hiding this comment

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

FYI @CaseyHillers for devicelab changes

LGTM

@fluttergithubbot fluttergithubbot merged commit 8158b47 into flutter:master Mar 31, 2021
@jiahaog jiahaog deleted the it-devicelab branch March 31, 2021 05:56
@jiahaog
Copy link
Member Author

jiahaog commented Mar 31, 2021

https://flutter-dashboard.appspot.com/#/build shows that the entire row for this commit is green, but I would have expect a new column to be added for this change, since new tests are added. Am I doing something wrong here? https://github.com/flutter/flutter/blob/master/dev/devicelab/README.md seems to be a bit out of date with prod_builders.json

@CaseyHillers
Copy link
Contributor

You'll need to update https://cs.opensource.google/flutter/infra/+/master:config/devicelab_config.star;bpv=1 to dedicate infrastructure to run these tasks. This PR only added UI to show the infrastructure.

Adding the task name to linux_tasks, mac_tasks will start running these.

@jiahaog
Copy link
Member Author

jiahaog commented Apr 1, 2021

Will do, opened https://github.com/flutter/infra/pull/401 for this!

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.

4 participants