Merge lp:~cypressyew/checkbox-legacy/gpu_test_fix1321995 into lp:checkbox-legacy

[Po-Hsu Lin]

Parse the kernel output message from dmesg instead of the kern.log.
To make sure this GPU hung issue is happened during this session.

Changes made to checkbox-legacy since this issue would cause false-positive result on SRU systems.

[Zygmunt Krynicki (zyga)]

+1

review: Approve

[Daniel Manrique (roadmr)]

The attempt to merge lp:~cypressyew/checkbox-legacy/gpu_test_fix1321995 into lp:checkbox-legacy failed. Below is the output from the failed tests.

[precise] Bringing VM 'up'
[precise] Unable to 'up' VM!
[precise] stdout: http://paste.ubuntu.com/8459982/
[precise] stderr: http://paste.ubuntu.com/8459983/
[precise] NOTE: unable to execute tests, marked as failed
[precise] Destroying failed VM to reclaim resources
[precise] Forcing shutdown of VM...
[precise] Destroying VM and associated drives...
[precise] Running cleanup tasks for 'shell' provisioner...
[precise] Running cleanup tasks for 'shell' provisioner...
[trusty] Bringing VM 'up'
[trusty] Unable to 'up' VM!
[trusty] stdout: http://paste.ubuntu.com/8460032/
[trusty] stderr: http://paste.ubuntu.com/8460033/
[trusty] NOTE: unable to execute tests, marked as failed
[trusty] Destroying failed VM to reclaim resources
[trusty] Forcing shutdown of VM...
[trusty] Destroying VM and associated drives...
[trusty] Running cleanup tasks for 'shell' provisioner...
[trusty] Running cleanup tasks for 'shell' provisioner...

[Daniel Manrique (roadmr)]

Reapproving to merge now that I fixed tarmac to use lxc for checkbox-legacy

review: Approve

=== modified file 'checkbox-old/CHANGELOG'
--- checkbox-old/CHANGELOG	2014-03-19 19:59:28 +0000
+++ checkbox-old/CHANGELOG	2014-05-22 06:17:51 +0000
@@ -11,6 +11,8 @@
   * Move the volume key test to right after the mute key test.
   * scripts/graphics_driver: added a new pattern for parsing Intel driver on
     12.04.4 (LP: #1279222) 
+  * scripts/gpu_test: parse the "gpu hung" pattern from dmesg instead of the
+    kern.log (LP: #1321995)
 
   [ Jeff Lane ]
   * scripts/network: fail threshold for iperf is now an option and defaults to

=== modified file 'checkbox-old/scripts/gpu_test'
--- checkbox-old/scripts/gpu_test	2013-10-09 15:50:42 +0000
+++ checkbox-old/scripts/gpu_test	2014-05-22 06:17:51 +0000
@@ -140,11 +140,10 @@
 
 def check_gpu(log=None):
     if not log:
-        log = '/var/log/kern.log'
-    with open(log, 'rb') as f:
-        if re.findall(r'gpu\s+hung', str(f.read()), flags=re.I):
-            print("GPU hung Detected")
-            return 1
+        log = subprocess.check_output(['dmesg'], universal_newlines=True)
+    if re.findall(r'gpu\s+hung', log, flags=re.I):
+        print("GPU hung Detected")
+        return 1
 
 
 def main():