Merge lp:~stub/launchpad/kill-harder into lp:launchpad
Proposed by
Stuart Bishop
Status: | Merged |
---|---|
Merged at revision: | not available |
Proposed branch: | lp:~stub/launchpad/kill-harder |
Merge into: | lp:launchpad |
Diff against target: | None lines |
To merge this branch: | bzr merge lp:~stub/launchpad/kill-harder |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Eleanor Berger (community) | Approve | ||
Review via email:
|
This proposal supersedes a proposal from 2009-09-10.
To post a comment you must log in.
Addresses Bug #307447
If bin/killservice fails to kill a service with a SIGTERM, retry again with a SIGKILL.
To test, run 'make run' in a terminal. Then, it a different terminal, run 'bin/killservice librarian'. This will test the entire code path due to the way the librarian is spawned. Then, run 'bin/killservice launchpad' to demonstrate the normal code path where the process shutsdown normally.
$ bin/killservice librarian
2009-09-10 11:13:58 INFO Killing librarian (31210)
2009-09-10 11:14:18 WARNING SIGTERM failed to kill librarian (31210). Trying SIGKILL
2009-09-10 11:14:38 ERROR SIGKILL didn't terminate librarian (31210)
$ bin/killservice launchpad
2009-09-10 11:14:47 INFO Killing launchpad (31200)