Merge "Give helpful error in tests if strace is missing"
This commit is contained in:
commit
9ec54a3f27
@ -23,6 +23,7 @@ and spinning down the servers.
|
|||||||
|
|
||||||
import atexit
|
import atexit
|
||||||
import datetime
|
import datetime
|
||||||
|
import errno
|
||||||
import logging
|
import logging
|
||||||
import os
|
import os
|
||||||
import platform
|
import platform
|
||||||
@ -857,7 +858,14 @@ class FunctionalTest(test_utils.BaseTestCase):
|
|||||||
trace = f.pid_file.replace('.pid', '.trace')
|
trace = f.pid_file.replace('.pid', '.trace')
|
||||||
if self.tracecmd:
|
if self.tracecmd:
|
||||||
cmd = '%s -p %d -o %s' % (self.tracecmd, pid, trace)
|
cmd = '%s -p %d -o %s' % (self.tracecmd, pid, trace)
|
||||||
execute(cmd, raise_error=False, expect_exit=False)
|
try:
|
||||||
|
execute(cmd, raise_error=False, expect_exit=False)
|
||||||
|
except OSError as e:
|
||||||
|
if e.errno == errno.ENOENT:
|
||||||
|
raise RuntimeError('No executable found for "%s" '
|
||||||
|
'command.' % self.tracecmd)
|
||||||
|
else:
|
||||||
|
raise
|
||||||
time.sleep(0.5)
|
time.sleep(0.5)
|
||||||
if os.path.exists(trace):
|
if os.path.exists(trace):
|
||||||
msg += ('\n%s:\n%s\n' % (self.tracecmd,
|
msg += ('\n%s:\n%s\n' % (self.tracecmd,
|
||||||
|
Loading…
Reference in New Issue
Block a user