Skip to content

Conversation

@zanderso
Copy link
Member

Reverts #122437

This is causing several Android devicelab tests to fail on the first attempt, causing them to be marked flaky. The cause appears to be unexpected output on stderr. See: https://ci.chromium.org/ui/p/flutter/builders/prod/Mac_arm64_android%20run_release_test/5390/overview

@flutter-dashboard flutter-dashboard bot added d: examples Sample code and demos framework flutter/packages/flutter repository. See also f: labels. f: integration_test The flutter/packages/integration_test plugin c: contributor-productivity Team-specific productivity, code health, technical debt. labels Apr 12, 2023
@zanderso
Copy link
Member Author

cc @micaelcid

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.

LGTM

@zanderso zanderso added the autosubmit Merge PR when tree becomes green via auto submit App label Apr 12, 2023
@zanderso zanderso merged commit cc2a045 into master Apr 12, 2023
@zanderso zanderso deleted the revert-122437-master branch April 12, 2023 01:55
@micaelcid
Copy link
Contributor

micaelcid commented Apr 12, 2023

In this case scenario, should we escalate this problem to an infrastructure team or someone else? I'm uncertain if I'm doing something wrong or if the tests' infrastructure should be upgraded along with Android test dependencies. 🤔

I mean, this was my second consecutive attempt to get these integration_test changes merged into master that got reverted, even though the PR pipeline didn't catch anything. I'm new to open-sourcing and I confess I feel a little bit lost in moments like these 😅

cc @zanderso

@zanderso
Copy link
Member Author

@micaelcid The error message in the logs that broke the tests is the following:

[2023-04-11 20:02:04.216019] [STDOUT] run:stderr: Note: /opt/s/w/ir/x/w/recipe_cleanup/tmpcgyxdgaw/flutter sdk/packages/integration_test/android/src/main/java/dev/flutter/plugins/integration_test/FlutterTestRunner.java uses or overrides a deprecated API.
[2023-04-11 20:02:04.218200] [STDOUT] run:stderr: Note: Recompile with -Xlint:deprecation for details.

Unfortunately, I'm not familiar with the change that you're trying to land, but I can tell you that the file it is complaining about is here: https://github.com/flutter/flutter/blob/master/packages/integration_test/android/src/main/java/dev/flutter/plugins/integration_test/FlutterTestRunner.java.

I suspect that the dependency that you are updating requires a change in that file to fix the error. @reidbaker may have more specific advice about how to proceed.

engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Apr 12, 2023
exaby73 pushed a commit to NevercodeHQ/flutter that referenced this pull request Apr 17, 2023
…24644)

Reverts flutter#122437

This is causing several Android devicelab tests to fail on the first
attempt, causing them to be marked flaky. The cause appears to be
unexpected output on stderr. See:
https://ci.chromium.org/ui/p/flutter/builders/prod/Mac_arm64_android%20run_release_test/5390/overview
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

autosubmit Merge PR when tree becomes green via auto submit App c: contributor-productivity Team-specific productivity, code health, technical debt. d: examples Sample code and demos f: integration_test The flutter/packages/integration_test plugin framework flutter/packages/flutter repository. See also f: labels.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants