RepoMirrors/ceph
1
0
Fork 0
mirror of https://github.com/ceph/ceph synced 2024-12-14 07:25:50 +00:00
Code Issues Packages Projects Releases Wiki Activity

Fix bug that thought all >1 node clusters always had core dumps.

Browse Source 
Accidentally shared the stdout between all the runs.
This commit is contained in:
Tommi Virtanen 2011-06-20 13:18:01 -07:00
parent 6a32f723a3
commit bc8cc868f9
1 changed files with 12 additions and 13 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

25
teuthology/task/internal.py
View File

@ -171,16 +171,15 @@ def coredump(ctx, config):
# set success=false if the dir is still there = coredumps were
# seen
processes = ctx.cluster.run(
args=[
'if', 'test', '!', '-e', '/tmp/cephtest/archive/coredump', run.Raw(';'), 'then',
'echo', 'OK', run.Raw(';'),
'fi',
],
wait=False,
stdout=StringIO(),
)
run.wait(processes)
if any(r.stdout.getvalue() != 'OK' for r in processes):
log.warning('Found coredumps, flagging run as failed.')
ctx.summary['success'] = False
for remote in ctx.cluster.remotes.iterkeys():
r = remote.run(
args=[
'if', 'test', '!', '-e', '/tmp/cephtest/archive/coredump', run.Raw(';'), 'then',
'echo', 'OK', run.Raw(';'),
'fi',
],
stdout=StringIO(),
)
if r.stdout.getvalue() != 'OK\n':
log.warning('Found coredumps on %s, flagging run as failed', remote)
ctx.summary['success'] = False