Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Two benchmarks that specify the same device in .ci.yaml are running on different devices. #105314

Closed
zanderso opened this issue Jun 3, 2022 · 4 comments · Fixed by #105322
Closed
Assignees
Labels
P1 High-priority issues at the top of the work list team-infra Owned by Infrastructure team

Comments

@zanderso
Copy link
Member

zanderso commented Jun 3, 2022

In the.ci.yaml file Linux_samsung_s10 new_gallery__transition_perf and Linux_samsung_s10 new_gallery_impeller__transition_perf both specify tags ["devicelab", "android", "linux", "samsung", "s10"], which I would have guessed would mean that both would be running on Samsung S10 devices.

However, looking at both SkiaPerf and at the pages for the builders, they are running on different device types.

For example here, the impeller version ran on flutter-devicelab-linux-18, which has device type SM-A025V.

But here, the non-Impeller version ran flutter-devicelab-linux-20, which has device type SM-G973U1.

We are going to be able to need to compare Impeller to non-Impeller across a range of benchmarks and devices, so I'd really like to be 100% sure about the devices that we're running on. How can I do that?

@keyonghan

@zanderso zanderso added team-infra Owned by Infrastructure team P1 High-priority issues at the top of the work list labels Jun 3, 2022
@zanderso zanderso added this to New in Infra Ticket Queue via automation Jun 3, 2022
@keyonghan
Copy link
Contributor

We have been relying on device_os dimension for task scheduling, but it turns out Samsung A02 and Samsung S10 may share the same device_os (ro.build.id):

testbed     device_os           device_type
linux-18    RP1A.200720.012     SM-A025V
linux-20    RP1A.200720.012     SM-G973U1
linux-26    SP1A.210812.016     SM-G973U1

But device_type is consistent to distinguish A02 from S10.

@zanderso Does it make sense to switch to device_type or do we not care the device_os?

@keyonghan keyonghan moved this from New to Triaged in Infra Ticket Queue Jun 3, 2022
@zanderso
Copy link
Member Author

zanderso commented Jun 3, 2022

For the purposes of benchmarking, the most important distinguishing feature is the device type.

@keyonghan keyonghan self-assigned this Jun 3, 2022
@keyonghan keyonghan moved this from Triaged to In progress in Infra Ticket Queue Jun 3, 2022
@keyonghan
Copy link
Contributor

#105322 to fix.

Infra Ticket Queue automation moved this from In progress to Done Jun 3, 2022
@github-actions
Copy link

This thread has been automatically locked since there has not been any recent activity after it was closed. If you are still experiencing a similar issue, please open a new bug, including the output of flutter doctor -v and a minimal reproduction of the issue.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jun 17, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
P1 High-priority issues at the top of the work list team-infra Owned by Infrastructure team
Projects
No open projects
Archived in project
Development

Successfully merging a pull request may close this issue.

2 participants