commit 6fae53c0449d05b9fbc9763a220e4bc1e39930cb
author Wentong Wu <wentong.wu@intel.com> Wed Jun 24 22:55:59 2020 +0800
committer Anas Nashif <anas.nashif@intel.com> Thu Jun 25 11:05:29 2020 -0400
tree 816feb718ab1de49d082b54fad1681d5a954f742
parent 1105aa521c298d74427e35d4cef27b501df40b6b

sanitycheck: handle timeouts correctly with qemu

In some cases we were not handling return code correctly and evaluating
timeouts as a pass. Report failure correctly.

Fixes #26065

Signed-off-by: Wentong Wu <wentong.wu@intel.com>

scripts/sanity_chk/sanitylib.py

diff --git a/scripts/sanity_chk/sanitylib.py b/scripts/sanity_chk/sanitylib.py
index 4e57633..90c0785 100644
--- a/scripts/sanity_chk/sanitylib.py
+++ b/scripts/sanity_chk/sanitylib.py
@@ -898,7 +898,10 @@
                     except ProcessLookupError:
                         pass
                     proc.wait()
-                    self.returncode = 0
+                    if harness.state == "passed":
+                        self.returncode = 0
+                    else:
+                        self.returncode = proc.returncode
                 else:
                     proc.terminate()
                     proc.kill()