diff options
| author | Markus Lehtonen <markus.lehtonen@linux.intel.com> | 2016-08-29 22:48:31 +0300 |
|---|---|---|
| committer | Richard Purdie <richard.purdie@linuxfoundation.org> | 2016-09-03 09:58:42 +0100 |
| commit | 7d77c02401cdc1925457d9dc3f6ed3bcbea14ced (patch) | |
| tree | 016d81f8454b5a78ee1373d0cd22f6964450d9c7 | |
| parent | c39db4bc4501c6e551dd2f2a299de76e9124f94f (diff) | |
| download | poky-7d77c02401cdc1925457d9dc3f6ed3bcbea14ced.tar.gz | |
oeqa.buildperf: include commands log file name in results.json
(From OE-Core rev: b22a71cf3a53a33763ff02608119d2c73cbde006)
Signed-off-by: Markus Lehtonen <markus.lehtonen@linux.intel.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
| -rw-r--r-- | meta/lib/oeqa/buildperf/base.py | 14 |
1 files changed, 9 insertions, 5 deletions
diff --git a/meta/lib/oeqa/buildperf/base.py b/meta/lib/oeqa/buildperf/base.py index 0b2c0f8e73..be3a946ddf 100644 --- a/meta/lib/oeqa/buildperf/base.py +++ b/meta/lib/oeqa/buildperf/base.py | |||
| @@ -220,6 +220,8 @@ class BuildPerfTestResult(unittest.TextTestResult): | |||
| 220 | 'status': status, | 220 | 'status': status, |
| 221 | 'start_time': test.start_time, | 221 | 'start_time': test.start_time, |
| 222 | 'elapsed_time': test.elapsed_time, | 222 | 'elapsed_time': test.elapsed_time, |
| 223 | 'cmd_log_file': os.path.relpath(test.cmd_log_file, | ||
| 224 | self.out_dir), | ||
| 223 | 'measurements': test.measurements} | 225 | 'measurements': test.measurements} |
| 224 | results['tests'] = tests | 226 | results['tests'] = tests |
| 225 | 227 | ||
| @@ -312,6 +314,10 @@ class BuildPerfTestCase(unittest.TestCase): | |||
| 312 | def out_dir(self): | 314 | def out_dir(self): |
| 313 | return os.path.join(self.base_dir, self.name) | 315 | return os.path.join(self.base_dir, self.name) |
| 314 | 316 | ||
| 317 | @property | ||
| 318 | def cmd_log_file(self): | ||
| 319 | return os.path.join(self.out_dir, 'commands.log') | ||
| 320 | |||
| 315 | def setUp(self): | 321 | def setUp(self): |
| 316 | """Set-up fixture for each test""" | 322 | """Set-up fixture for each test""" |
| 317 | if self.build_target: | 323 | if self.build_target: |
| @@ -328,9 +334,8 @@ class BuildPerfTestCase(unittest.TestCase): | |||
| 328 | """Run a command and log it's output""" | 334 | """Run a command and log it's output""" |
| 329 | cmd_str = cmd if isinstance(cmd, str) else ' '.join(cmd) | 335 | cmd_str = cmd if isinstance(cmd, str) else ' '.join(cmd) |
| 330 | log.info("Logging command: %s", cmd_str) | 336 | log.info("Logging command: %s", cmd_str) |
| 331 | cmd_log = os.path.join(self.out_dir, 'commands.log') | ||
| 332 | try: | 337 | try: |
| 333 | with open(cmd_log, 'a') as fobj: | 338 | with open(self.cmd_log_file, 'a') as fobj: |
| 334 | runCmd2(cmd, stdout=fobj) | 339 | runCmd2(cmd, stdout=fobj) |
| 335 | except CommandError as err: | 340 | except CommandError as err: |
| 336 | log.error("Command failed: %s", err.retcode) | 341 | log.error("Command failed: %s", err.retcode) |
| @@ -368,9 +373,8 @@ class BuildPerfTestCase(unittest.TestCase): | |||
| 368 | cmd_str = cmd if isinstance(cmd, str) else ' '.join(cmd) | 373 | cmd_str = cmd if isinstance(cmd, str) else ' '.join(cmd) |
| 369 | log.info("Timing command: %s", cmd_str) | 374 | log.info("Timing command: %s", cmd_str) |
| 370 | data_q = SimpleQueue() | 375 | data_q = SimpleQueue() |
| 371 | cmd_log = os.path.join(self.out_dir, 'commands.log') | ||
| 372 | try: | 376 | try: |
| 373 | with open(cmd_log, 'a') as fobj: | 377 | with open(self.cmd_log_file, 'a') as fobj: |
| 374 | proc = Process(target=_worker, args=(data_q, cmd,), | 378 | proc = Process(target=_worker, args=(data_q, cmd,), |
| 375 | kwargs={'stdout': fobj}) | 379 | kwargs={'stdout': fobj}) |
| 376 | proc.start() | 380 | proc.start() |
| @@ -380,7 +384,7 @@ class BuildPerfTestCase(unittest.TestCase): | |||
| 380 | raise data | 384 | raise data |
| 381 | except CommandError: | 385 | except CommandError: |
| 382 | log.error("Command '%s' failed, see %s for more details", cmd_str, | 386 | log.error("Command '%s' failed, see %s for more details", cmd_str, |
| 383 | cmd_log) | 387 | self.cmd_log_file) |
| 384 | raise | 388 | raise |
| 385 | etime = data['elapsed_time'] | 389 | etime = data['elapsed_time'] |
| 386 | 390 | ||
