Skip to content

Linux_android hot_mode_dev_cycle_linux__benchmark runs in presubmit, only one bot available in try pool #145242

Closed

Description

@jmagman

Linux_android hot_mode_dev_cycle_linux__benchmark runs on a linux-motog4 bot, and runs in presubmit.

flutter/.ci.yaml

Lines 2394 to 2401 in 55d8b20

# linux motog4 benchmark
- name: Linux_android hot_mode_dev_cycle_linux__benchmark
recipe: devicelab/devicelab_drone
timeout: 60
properties:
tags: >
["devicelab", "android", "linux"]
task_name: hot_mode_dev_cycle_linux__benchmark

However, there is only one of these bots in the try pool.
https://chromium-swarm.appspot.com/task?id=685d3bbc1eacaf10&o=true&w=true
https://chromium-swarm.appspot.com/botlist?f=os%3ALinux&f=pool%3Aluci.flutter.try&f=device_type%3Amsm8952
Screenshot 2024-03-15 at 4 26 11 PM

That means PRs are all bottlenecked on one bot. And if that bot dies all the PRs will be pending until it comes back up.

Either:

  1. Add more linux-motog4 bots to the try pool
  2. Stop running this test in presubmit

Activity

  1. ricardoamador commented on Mar 18, 2024

    @ricardoamador
    Contributor

    @yusuf-goog this is correct. I went and checked the other pools and there are 13 of these bots in postsubmit. Do you recall if there was a reason for this imbalance between the pools?

  2. added
    P1High-priority issues at the top of the work list
    triaged-infraTriaged by Infrastructure team
    triaged-iosTriaged by iOS platform team
    and removed
    triaged-iosTriaged by iOS platform team
    on Mar 18, 2024
  3. ricardoamador commented on Mar 18, 2024

    @ricardoamador
    Contributor

    The understanding was this bot was for testing out changes in the try pool and was not meant to be a blocker to presubmit changes. I'm going to move the bot to prod and change the target to prod as well.

    Switch target to prod: #145343
    Switch bot to prod: https://chrome-internal-review.googlesource.com/c/infradata/config/+/7078725

  4. jmagman commented on Mar 18, 2024

    @jmagman
    MemberAuthor

    Switch bot to prod: https://chrome-internal-review.googlesource.com/c/infradata/config/+/7078725

    Note this will likely need to be moved back if hot_mode_dev_cycle_linux__benchmark is ever marked flaky, and run in the staging pool.

  5. ricardoamador commented on Mar 18, 2024

    @ricardoamador
    Contributor

    @jmagman the flaky bot will take care of this if the test ever becomes flaky. 👍🏼

  6. yusuf-goog commented on Mar 19, 2024

    @yusuf-goog
    Contributor

    FYI @zanderso . The one linux-moto G4 in try has been moved out: https://chrome-internal-review.googlesource.com/c/infradata/config/+/7078725

    If you still want to run benchmarks that are non-blocking to the tree (since we do not have the required minimum capacity in try) we would need to move this testsetup back in.

  7. zanderso commented on Mar 20, 2024

    @zanderso
    Member

    I see that there's one Linux/MotoG4 set up in staging on flutter-devicelab-linux-14, so if one of the prod Linux/MotoG4 tests becomes flaky we will be okay there. There aren't currently enough devicelab bots of any configuration in the try pool to run in presubmit reliably on every PR, so moving this test off of presubmit (that is, out of the try pool) makes sense to me. After the devicelab moves and the new low-end device is hooked up, let's revisit whether we can get enough devicelab bots of some configuration to run in presub.

  8. ricardoamador commented on Mar 20, 2024

    @ricardoamador
    Contributor

    @zanderso thanks for clarifying Zach, I will open a tracking issue for this so we can make sure to circle back.

  9. ricardoamador commented on Mar 20, 2024

    @ricardoamador
    Contributor

    Created #145480 for tracking purposes.

  10. moved this from Fixed to Verified in [DEPRECATED] Infra Ticket Queueon Mar 25, 2024
  11. github-actions commented on Apr 3, 2024

    @github-actions

    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.

  12. locked as resolved and limited conversation to collaborators on Apr 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

P1High-priority issues at the top of the work listinfra: device labInfrastructure device lab problemsteam-infraOwned by Infrastructure teamtriaged-infraTriaged by Infrastructure team

Type

No type

Projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions