BIND 10 #134: xfrout not killed by SIGTERM

[BIND 10 Development]

#134: xfrout not killed by SIGTERM
--------------------------+-------------------------------------------------
 Reporter:  jreed         |        Owner:     
     Type:  defect        |       Status:  new
 Priority:  major         |    Milestone:     
Component:  Unclassified  |   Resolution:     
 Keywords:                |    Sensitive:  0  
--------------------------+-------------------------------------------------

Comment(by shane):

 Replying to [comment:3 jreed]:
 > Replying to [comment:2 zhanglikun]:
 >
 > > So the easy way to fix this is: Make boss sleep more seconds after
 send out "shutdown" message?
 >
 > The BSD reboot(8) since at least 1990 waits 5 seconds after SIGTERM
 before doing a SIGKILL. I think that 5 seconds is good. If it ever gets to
 SIGKILL then we have a problem and should report that.

 Well... we have a few things to do to make this as good as possible.

 First, xfrout should implement shutdown.

 Second, bind10 can wait a much longer time before proceeding from one
 severity to another.

 To do this "properly", bind10 should do something like this:

   * Send a shutdown request to all processes
   * Wait for processes to die
   * When they all die, exit
   * If more than X seconds pass, send SIGTERM to all processes
   * Wait for processes to die
   * When they all die, exit
   * If more than X seconds pass, send SIGKILL to all processes
   * Wait for processes to die

 This would both speed up the usual case (right now we wait 0.2 seconds
 whether we need to or not) and slow down the extreme case (since we'll be
 waiting SECONDS instead of tenths of seconds). I tend to think X should be
 1 or 2 - nothing should take that long to stop unless things are broken.

 Something we can also do is create a "fast-shutdown" option. This might
 change X from 1 or 2 to 0.1 like we have now (although still exiting
 faster under normal circumstances).

 Does this make sense?

-- 
Ticket URL: <http://bind10.isc.org/ticket/134#comment:4>
BIND 10 Development <http://bind10.isc.org>
BIND 10 Development