Project

General

Profile

Actions

Bug #14834

closed

test_pidfile.sh leave two osd daemon running after unittest

Added by shun song about 8 years ago. Updated about 8 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
Category:
-
Target version:
-
% Done:

0%

Source:
other
Tags:
Backport:
hammer
Regression:
No
Severity:
3 - minor
Reviewed:
Affected Versions:
ceph-qa-suite:
Pull request ID:
Crash signature (v1):
Crash signature (v2):

Description

not killed two daemons comes out at line 90 and line 73 in test_pidfile.sh. The detail description is seen at https://github.com/ceph/ceph/pull/7463.

solution is attached by file, preview as follows:

diff test/test_pidfile.sh.old test/test_pidfile.sh.new 
34a35,38

kill_daemons $dir TERM osd.0 || return 1
! test -f $dir/osd.0.pid || return 1
./ceph-osd -i 0 --osd-data=$dir/0 --osd-journal=$dir/0/journal--pid-file= --daemonize=$RUNID || { teardown_unexist_pidfile $dir $RUNID; return 1; }
! test -f $dir/osd.0.pid || return 1

94a99

cp $dir/osd.0.pid $dir/osd.0.pid.old


Files

test_pidfile.sh (3.33 KB) test_pidfile.sh shun song, 02/22/2016 02:39 AM
Actions #1

Updated by Loïc Dachary about 8 years ago

  • Status changed from New to Fix Under Review
Actions #2

Updated by Loïc Dachary about 8 years ago

  • Status changed from Fix Under Review to Resolved
Actions

Also available in: Atom PDF