Skip to content
Browse files

- Fix bug where stopping process would cause process output that hap…

…pened

    after the stop request was issued to be lost.  See
    #11.

Closes #11.
  • Loading branch information...
1 parent 7e4ab0a commit e19cbc185dfad045c8775750d36ab8ceed4c4dfb @mcdonc mcdonc committed Mar 30, 2011
Showing with 4 additions and 2 deletions.
  1. +4 −0 CHANGES.txt
  2. +0 −1 src/supervisor/process.py
  3. +0 −1 src/supervisor/tests/test_process.py
View
4 CHANGES.txt
@@ -21,6 +21,10 @@ Next release
``[supervisord]`` section had no effect. Thanks to Wyatt Baldwin
for a patch.
+ - Fix bug where stopping process would cause process output that happened
+ after the stop request was issued to be lost. See
+ https://github.com/Supervisor/supervisor/issues/11.
+
3.0a9 (2010-08-13)
- Use rich comparison methods rather than __cmp__ to sort process
View
1 src/supervisor/process.py
@@ -341,7 +341,6 @@ def _spawn_as_child(self, filename, argv):
def stop(self):
""" Administrative stop """
- self.drain()
self.administrative_stop = 1
return self.kill(self.config.stopsignal)
View
1 src/supervisor/tests/test_process.py
@@ -608,7 +608,6 @@ def test_stop(self):
'signal SIGTERM')
self.assertEqual(instance.killing, 1)
self.assertEqual(options.kills[11], signal.SIGTERM)
- self.assertEqual(dispatcher.write_event_handled, True)
def test_give_up(self):
options = DummyOptions()

0 comments on commit e19cbc1

Please sign in to comment.
Something went wrong with that request. Please try again.