Skip to content

Windows tool_integration_tests_1_6 flakes on android_plugin_example_app_build_test #98085

Description

@keyonghan

(part of #97036)

Hit one flake today:

12:30 +22 ~2 -1: test\integration.shard\android_plugin_example_app_build_test.dart: FFI plugin example can be built using current Flutter Gradle plugin [E]                                            
  Expected: a file system entity that exists
    Actual: LocalFile:<LocalFile: 'C:\b\s\w\ir\x\t\flutter_flutter_ffi_plugin_test.aae6b239\plugin_ffi_test\example\build\app\outputs\flutter-apk\app-release.apk'>
     Which: does not exist
  
  package:test_api                                                          expect
  test\integration.shard\android_plugin_example_app_build_test.dart 79:5    main.testPlugin
  test\integration.shard\android_plugin_example_app_build_test.dart 26:26   main.testPlugin
  test\integration.shard\android_plugin_example_app_build_test.dart 131:11  main.<fn>
  test\integration.shard\android_plugin_example_app_build_test.dart 130:7   main.<fn>
  test\src\common.dart 166:18                                               test.<fn>
  test\src\common.dart 161:5                                                test.<fn>

Activity

  1. added
    toolAffects the "flutter" command-line tool. See also t: labels.
    on Feb 9, 2022
  2. keyonghan commented on Feb 9, 2022

    @keyonghan
    ContributorAuthor
  3. zanderso commented on Feb 9, 2022

    @zanderso
    Member

    Routing to @dcharkes who added the test a couple weeks back. My suggestion for debugging the flake would be to land more debugging output from the testPlugin plugin method in that file. In particular, if a call to processManager.runSync() fails, it should print the exit code, stdout, and stderr from the subprocess.

  4. keyonghan commented on Apr 8, 2022

    @keyonghan
    ContributorAuthor
  5. zanderso commented on Apr 8, 2022

    @zanderso
    Member

    @dcharkes ptal.

  6. dcharkes commented on Apr 11, 2022

    @dcharkes
    Contributor

    This time it is the platform-channel-plugin that is failing, not the FFI plugin.

    And it is failing in the flutter invocation which is not run for the FFI plugins, so my last PR did not add extra prints there.

    Here's a PR which adds prints to the remaining invocations:

  7. keyonghan commented on Jul 22, 2022

    @keyonghan
    ContributorAuthor
  8. added
    c: flakeTests that sometimes, but not always, incorrectly pass
    on Jul 22, 2022
  9. zanderso commented on Jul 22, 2022

    @zanderso
    Member

    @keyonghan both of those failures seem to be saying that the NDK on the bot is malformed. In one case the zip archive failed to be extracted correctly, and in the other, it was missing a file.

  10. added
    team-infraOwned by Infrastructure team
    and removed
    toolAffects the "flutter" command-line tool. See also t: labels.
    on Jul 22, 2022
  11. keyonghan commented on Jul 22, 2022

    @keyonghan
    ContributorAuthor

    @keyonghan both of those failures seem to be saying that the NDK on the bot is malformed. In one case the zip archive failed to be extracted correctly, and in the other, it was missing a file.

    Thanks @zanderso for the triage.
    It is indeed with the bot. Both failures happened on https://chromium-swarm.appspot.com/bot?id=flutter-prod-windows-us-central1-a-11-fv9g, which seemed have caused tests failure across other tests.
    I clicked STOP THE BOT GRACEFULLY, and this bot should not be assigned to any tasks for today. Additionally, all bots will be auto re-provisioned tomorrow

    On top of that, the original filed flaky issue is not reproducing.

    I will monitor this bot issue a bit before closing.

  12. self-assigned this
    on Jul 22, 2022
  13. keyonghan commented on Jul 27, 2022

    @keyonghan
    ContributorAuthor

    Closing as this didn't repro.

  14. github-actions commented on Aug 10, 2022

    @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.

  15. locked as resolved and limited conversation to collaborators on Aug 10, 2022
  16. added
    P0Critical issues such as a build break or regression
    and removed on Jun 28, 2023
  17. christopherfujino commented on Nov 7, 2023

    @christopherfujino
    Contributor
  18. dcharkes commented on Nov 8, 2023

    @dcharkes
    Contributor

    AGP downloads the NDK (the version it wants) on demand, so maybe some bits fell over on the network? I'm not really sure what to do about this.

    (Side note: With the native assets, --template=package_ffi instead of --template=plugin_ffi, flutter_tools controls what NDK it uses for native builds: the NDK inside the Android SDK detected by flutter doctor. For our bots that means using the NDK from the CIPD package. So hopefully the new way of doing things will not have these kind of install-via-the-network flakes.)

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

Metadata

Metadata

Assignees

Labels

P0Critical issues such as a build break or regressionc: flakeTests that sometimes, but not always, incorrectly passteam-infraOwned by Infrastructure team

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions