Make output of all tests verbose
Re-enable more verbose output across all bots.
There is no good reason why we shouldn't collect additional
information that helps triaging failure.
Some of the current output might have to be expanded down the road.
Bug: skia:
Change-Id: I5ff89a803f1e390bbd859756cba53a359d2801ba
Reviewed-on: https://skia-review.googlesource.com/45140
Reviewed-by: Eric Boren <borenet@google.com>
Reviewed-by: Ben Wagner <benjaminwagner@google.com>
Reviewed-by: Kevin Lubick <kjlubick@google.com>
Commit-Queue: Stephan Altmueller <stephana@google.com>
diff --git a/infra/bots/recipes/test.py b/infra/bots/recipes/test.py
index 5192bcb..494296b 100644
--- a/infra/bots/recipes/test.py
+++ b/infra/bots/recipes/test.py
@@ -657,28 +657,8 @@
or 'Win8-MSVC-ShuttleB' in bot):
args.append('--noRAW_threading')
- # Some people don't like verbose output.
- verbose = False
-
- if 'Intel' in bot and api.vars.is_linux and not 'Vulkan' in bot:
- # TODO(dogben): Track down what's causing bots to die.
- verbose = True
-
- if 'Valgrind' in bot and 'PreAbandonGpuContext' in bot:
- verbose = True
-
- if 'NexusPlayer' in bot and 'CPU' in bot:
- # The Nexus Player's image decoding tests are slow enough that swarming
- # times it out for not printing anything frequently enough. --verbose
- # makes dm print something every time we start or complete a task.
- verbose = True
-
- if 'Android' in bot or 'iOS' in bot:
- # Enable verbose output on mobile platforms.
- verbose = True
-
- if verbose:
- args.append('--verbose')
+ # Let's make all bots produce verbose output by default.
+ args.append('--verbose')
return args