Skip to content
This repository was archived by the owner on Feb 25, 2025. It is now read-only.

Conversation

@chinmaygarde
Copy link
Member

We print stdout and stderr only from successful test or benchmark runs. Because
of the raised exception, a non-zero exit code would gobble up output from these
runs. This is actually the situation where this information would be most
useful. Print all the information we have before raising the exception.

…_tests.py`.

We print stdout and stderr only from successful test or benchmark runs. Because
of the raised exception, a non-zero exit code would gobble up output from these
runs. This is actually the situation where this information would be most
useful. Print all the information we have before raising the exception.
@chinmaygarde
Copy link
Member Author

chinmaygarde commented Apr 25, 2021

Ran into this while debugging the presubmit failures in #25749. Because of the silent failure, I mistook the failure for a flake. Now, the output would be:

!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!

Failed Command:

/Users/chinmaygarde/VersionControlled/engine/src/out/host_release/flutter_tester --force-multithreading --disable-observatory --use-test-fonts /Users/chinmaygarde/VersionControlled/engine/src/out/paragraph_builder_test.dart.kernel.dill

Exit Code: -6

STDERR: 
[FATAL:flutter/runtime/dart_vm_initializer.cc(22)] Error while initializing the Dart VM: Precompiled runtime requires a precompiled snapshot

!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!

@chinmaygarde chinmaygarde added the waiting for tree to go green This PR is approved and tested, but waiting for the tree to be green to land. label Apr 25, 2021
@fluttergithubbot fluttergithubbot merged commit 71fc4c1 into flutter:master Apr 25, 2021
GuoXi pushed a commit to GuoXi/engine that referenced this pull request Apr 25, 2021
@chinmaygarde chinmaygarde deleted the logs_from_test_failures branch April 25, 2021 22:41
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Apr 26, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

cla: yes waiting for tree to go green This PR is approved and tested, but waiting for the tree to be green to land.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants