BIND 10 #565: Boss should start/stop b10-auth and b10-recurse without restart

BIND 10 Development do-not-reply at isc.org
Wed Feb 23 14:46:37 UTC 2011


#565: Boss should start/stop b10-auth and b10-recurse without restart
-------------------------------------+-------------------------------------
                 Reporter:  vorner   |                Owner:  jinmei
                     Type:  defect   |               Status:  reviewing
                 Priority:  major    |            Milestone:  A-Team-
                Component:  Boss of  |  Sprint-20110223
  BIND                               |           Resolution:
                 Keywords:           |            Sensitive:  0
Estimated Number of Hours:  3.0      |  Add Hours to Ticket:  0
                Billable?:  1        |          Total Hours:  0
                Internal?:  0        |
-------------------------------------+-------------------------------------
Changes (by vorner):

 * owner:  vorner => jinmei


Comment:

 Hello

 Replying to [comment:5 jinmei]:
 > While looking at the code, however, some fundamental architecture
 > [ ... ]
 > These are probably not an issue specific to this patch but rather a
 > larger architectural one for the BIND10 process model, though.  To
 > make it really production ready, I guess we need to carefully design
 > state management of the subprocesses so that the questions like the
 > above ones are clarified.

 I agree. I think boss is one of the parts we want to have a closer look
 and refactor it. We need the dependencies there, and more general system
 of starting things.

 >  - In StartAllProcessesBob, we probably don't need all of the stop_xxx
 >    (e.g., stop_ccsession())

 I want to have them in case someone starts using stop_ something that is
 not used now and forgets to update the tests. This way, all possible ones
 are overriden.

 > I would regard this change as 'func', although I don't oppose if you
 > prefer calling it a bug.

 OK, func is probably OK as well.

 I should have addressed everything mentioned in the comments.

 Thanks

-- 
Ticket URL: <https://bind10.isc.org/ticket/565#comment:7>
BIND 10 Development <http://bind10.isc.org>
BIND 10 Development


More information about the bind10-tickets mailing list