diff options
author | Markus Lehtonen <markus.lehtonen@linux.intel.com> | 2017-01-27 16:54:53 +0200 |
---|---|---|
committer | Richard Purdie <richard.purdie@linuxfoundation.org> | 2017-02-15 20:06:41 -0800 |
commit | fcd28fd50f6f3768c41f1db184f55cd439699bd7 (patch) | |
tree | 9d182fdd08f21048dabe1711575d772952075f76 /meta/lib/oeqa/buildperf/test_basic.py | |
parent | 8279d6f257440fdcc60d5668daa4081044370265 (diff) | |
download | poky-fcd28fd50f6f3768c41f1db184f55cd439699bd7.tar.gz |
oeqa/buildperf: don't archive stdout/stderr of commands
Stop capturing output of the shell commands into <test>/commands.log.
Redirecting output into a file prevented the unittest framework from
capturing it, causing useless errors (with empty output) like:
oeqa.utils.CommandError: Command '['bitbake', 'core-image-sato']'
returned non-zero exit status 1 with output:
In general, the console output of commands is only interesting when
something fails. Also, dropping the commands.log file is a huge saving
in disk space, and thus, repository size when results are archived in
Git.
(From OE-Core rev: e004664287ec03e7367a7bf553d9a3038444e82e)
Signed-off-by: Markus Lehtonen <markus.lehtonen@linux.intel.com>
Signed-off-by: Ross Burton <ross.burton@intel.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
Diffstat (limited to 'meta/lib/oeqa/buildperf/test_basic.py')
-rw-r--r-- | meta/lib/oeqa/buildperf/test_basic.py | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/meta/lib/oeqa/buildperf/test_basic.py b/meta/lib/oeqa/buildperf/test_basic.py index 47118306b7..1333407a65 100644 --- a/meta/lib/oeqa/buildperf/test_basic.py +++ b/meta/lib/oeqa/buildperf/test_basic.py | |||
@@ -38,8 +38,8 @@ class Test1P2(BuildPerfTestCase): | |||
38 | def test12(self): | 38 | def test12(self): |
39 | """Build virtual/kernel""" | 39 | """Build virtual/kernel""" |
40 | # Build and cleans state in order to get all dependencies pre-built | 40 | # Build and cleans state in order to get all dependencies pre-built |
41 | self.log_cmd_output(['bitbake', self.build_target]) | 41 | self.run_cmd(['bitbake', self.build_target]) |
42 | self.log_cmd_output(['bitbake', self.build_target, '-c', 'cleansstate']) | 42 | self.run_cmd(['bitbake', self.build_target, '-c', 'cleansstate']) |
43 | 43 | ||
44 | self.sync() | 44 | self.sync() |
45 | self.measure_cmd_resources(['bitbake', self.build_target], 'build', | 45 | self.measure_cmd_resources(['bitbake', self.build_target], 'build', |
@@ -74,7 +74,7 @@ class Test2(BuildPerfTestCase): | |||
74 | def test2(self): | 74 | def test2(self): |
75 | """Run core-image-sato do_rootfs with sstate""" | 75 | """Run core-image-sato do_rootfs with sstate""" |
76 | # Build once in order to populate sstate cache | 76 | # Build once in order to populate sstate cache |
77 | self.log_cmd_output(['bitbake', self.build_target]) | 77 | self.run_cmd(['bitbake', self.build_target]) |
78 | 78 | ||
79 | self.rm_tmp() | 79 | self.rm_tmp() |
80 | self.rm_cache() | 80 | self.rm_cache() |
@@ -106,8 +106,8 @@ class Test4(BuildPerfTestCase): | |||
106 | 106 | ||
107 | def test4(self): | 107 | def test4(self): |
108 | """eSDK metrics""" | 108 | """eSDK metrics""" |
109 | self.log_cmd_output("bitbake {} -c do_populate_sdk_ext".format( | 109 | self.run_cmd(['bitbake', '-c', 'do_populate_sdk_ext', |
110 | self.build_target)) | 110 | self.build_target]) |
111 | self.bb_vars = get_bb_vars(None, self.build_target) | 111 | self.bb_vars = get_bb_vars(None, self.build_target) |
112 | tmp_dir = self.bb_vars['TMPDIR'] | 112 | tmp_dir = self.bb_vars['TMPDIR'] |
113 | installer = os.path.join( | 113 | installer = os.path.join( |