diff options
| author | Chase Qi <chase.qi@linaro.org> | 2017-08-18 16:49:57 +0800 |
|---|---|---|
| committer | Chase Qi <chase.qi@linaro.org> | 2017-08-18 16:49:57 +0800 |
| commit | 5752d03e82202c8280133cfe945bd1e2ff4cb78f (patch) | |
| tree | b00838662e1f51c271109b8d6f2e18a29405bfb4 /automated/lib | |
| parent | a11b30a6e70ad1abbff2d823f901c8e652a3d387 (diff) | |
py_test_lib: add_result: don't print result line to stdout
Printing result lines to stdout could be expensive for test case like
CtsLibcoreTestCases which generate 40k+ result lines. It is a lot of
overhead. In LAVA, the printing will take more then 3 hours. Refer to
the below job.
https://lkft.validation.linaro.org/scheduler/job/19464
Test tool itself will print result line to stdout. In LAVA,
lava-test-case will send result lines to stdout. For local run,
test-runner will print the content of result.csv. So the printing in
add_result() is actually unnecessary.
Change-Id: I4eea7dcf1133f81205944ee51c8216b81afeb8b9
Signed-off-by: Chase Qi <chase.qi@linaro.org>
Diffstat (limited to 'automated/lib')
| -rwxr-xr-x | automated/lib/py_test_lib.py | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/automated/lib/py_test_lib.py b/automated/lib/py_test_lib.py index e8932bf..0ccd963 100755 --- a/automated/lib/py_test_lib.py +++ b/automated/lib/py_test_lib.py @@ -1,4 +1,3 @@ def add_result(result_file, result): - print(result) with open(result_file, 'a') as f: f.write('%s\n' % result) |