commit e0c95e54b8f061c83f793df87eebeea54fa1824a
author Kazuhiro Inaba <kinaba@google.com> Mon Oct 26 17:14:12 2020 +0900
committer Commit Bot <commit-bot@chromium.org> Wed Oct 28 09:44:09 2020 +0000
tree 2953a681ec242d997e972d54807a647c3b27e760
parent 78cd87d75ffe18ef7bdb44a9b5faddbacb27264a

tradefed_test: Clean up new result parser outputs.

Debug log is duplicated with the tradefed output.

BUG=b:151386998
TEST=none

Change-Id: I2f2d984cfa6786006a4b8f345d7b429ad3af1ca0
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/autotest/+/2497747
Tested-by: Kazuhiro Inaba <kinaba@chromium.org>
Auto-Submit: Kazuhiro Inaba <kinaba@chromium.org>
Commit-Queue: Kazuhiro Inaba <kinaba@chromium.org>
Commit-Queue: Jaydeep Mehta <jaydeepmehta@google.com>
Reviewed-by: Jaydeep Mehta <jaydeepmehta@google.com>

server/cros/tradefed/tradefed_utils.py

diff --git a/server/cros/tradefed/tradefed_utils.py b/server/cros/tradefed/tradefed_utils.py
index 1017621..8d18b79 100644
--- a/server/cros/tradefed/tradefed_utils.py
+++ b/server/cros/tradefed/tradefed_utils.py
@@ -128,10 +128,6 @@
                                 waived_count.get(test_name, 0) + 1)
                         else:
                             failed_tests.add(test_name)
-                            logging.debug('error message %s\n',
-                                      failed_message)
-                            logging.debug('>>>> stacktrace %s\n',
-                                      failed_stacktrace)
 
         # Check for test completion.
         for summary in root.iter('Summary'):
@@ -139,7 +135,7 @@
             modules_total = summary.get('modules_total')
 
         if failed_tests:
-            logging.error('Failed (but not waived) tests:\n %s',
+            logging.error('Failed (but not waived) tests:\n%s',
                           '\n'.join(sorted(failed_tests)))
 
         waived = []