make test: enhance core-file information

Change-Id: I1283960a9a49f6d70b9d7b7793cfb345c22ccdea
Signed-off-by: Klement Sekera <ksekera@cisco.com>
This commit is contained in:
Klement Sekera
2018-03-23 10:50:11 +01:00
committed by Dave Wallace
parent f0d0ddbff2
commit 9b6ece7952
2 changed files with 32 additions and 17 deletions

View File

@ -4,6 +4,7 @@ import sys
import traceback
from log import RED, single_line_delim, double_line_delim
from debug import spawn_gdb
from subprocess import check_output, CalledProcessError
class Hook(object):
@ -67,8 +68,17 @@ class PollHook(Hook):
open('%s/_core_handled' % self.testcase.tempdir, 'a').close()
spawn_gdb(self.testcase.vpp_bin, core_path, self.logger)
else:
self.logger.critical("Core file present, debug with: gdb %s %s" %
(self.testcase.vpp_bin, core_path))
self.logger.error("Core file present, debug with: gdb %s %s" %
(self.testcase.vpp_bin, core_path))
self.logger.error("Running `file %s':" % core_path)
try:
info = check_output(["file", core_path])
self.logger.error(info)
except CalledProcessError as e:
self.logger.error(
"Could not run `file' utility on core-file, "
"rc=%s" % e.returncode)
pass
def poll_vpp(self):
"""

View File

@ -12,6 +12,7 @@ from framework import VppTestRunner
from debug import spawn_gdb
from log import global_logger
from discover_tests import discover_tests
from subprocess import check_output, CalledProcessError
# timeout which controls how long the child has to finish after seeing
# a core dump in test temporary directory. If this is exceeded, parent assumes
@ -99,8 +100,8 @@ def run_forked(suite):
(last_test, last_test_temp_dir))
elif not child.is_alive():
fail = True
global_logger.critical("Child process unexpectedly died (last "
"test running was `%s' in `%s')!" %
global_logger.critical("Child python process unexpectedly died "
"(last test running was `%s' in `%s')!" %
(last_test, last_test_temp_dir))
elif last_test_temp_dir and last_test_vpp_binary:
core_path = "%s/core" % last_test_temp_dir
@ -111,8 +112,8 @@ def run_forked(suite):
if not os.path.isfile(
"%s/_core_handled" % last_test_temp_dir):
global_logger.critical(
"Child unresponsive and core-file exists in test "
"temporary directory!")
"Child python process unresponsive and core-file "
"exists in test temporary directory!")
fail = True
if fail:
@ -123,7 +124,7 @@ def run_forked(suite):
"test: %s -> %s" % (link_path, lttd))
try:
os.symlink(last_test_temp_dir, link_path)
except:
except Exception:
pass
api_post_mortem_path = "/tmp/api_post_mortem.%d" % vpp_pid
if os.path.isfile(api_post_mortem_path):
@ -135,6 +136,15 @@ def run_forked(suite):
if os.path.isfile(core_path):
global_logger.error("Core-file exists in test temporary "
"directory: %s!" % core_path)
global_logger.debug("Running `file %s':" % core_path)
try:
info = check_output(["file", core_path])
global_logger.debug(info)
except CalledProcessError as e:
global_logger.error(
"Could not run `file' utility on core-file, "
"rc=%s" % e.returncode)
pass
if debug_core:
spawn_gdb(last_test_vpp_binary, core_path,
global_logger)
@ -151,19 +161,16 @@ if __name__ == '__main__':
try:
verbose = int(os.getenv("V", 0))
except:
except ValueError:
verbose = 0
default_test_timeout = 600 # 10 minutes
try:
test_timeout = int(os.getenv("TIMEOUT", default_test_timeout))
except:
except ValueError:
test_timeout = default_test_timeout
try:
debug = os.getenv("DEBUG")
except:
debug = None
debug = os.getenv("DEBUG")
s = os.getenv("STEP", "n")
step = True if s.lower() in ("y", "yes", "1") else False
@ -184,10 +191,8 @@ if __name__ == '__main__':
discover_tests(d, cb)
try:
retries = int(os.getenv("RETRIES"))
except:
retries = 0
if retries is None:
retries = int(os.getenv("RETRIES", 0))
except ValueError:
retries = 0
attempts = retries + 1
if attempts > 1: