BIND 10 #2020: b10-ddns should get a list of secondary zones from zonemgr

[BIND 10 Development]

#2020: b10-ddns should get a list of secondary zones from zonemgr
-------------------------------------+-------------------------------------
                   Reporter:         |                 Owner:  jinmei
  jinmei                             |                Status:  reviewing
                       Type:  task   |             Milestone:
                   Priority:         |  Sprint-20120619
  medium                             |            Resolution:
                  Component:  DDNS   |             Sensitive:  0
                   Keywords:         |           Sub-Project:  DNS
            Defect Severity:  N/A    |  Estimated Difficulty:  0
Feature Depending on Ticket:  DDNS   |           Total Hours:  0
        Add Hours to Ticket:  0      |
                  Internal?:  0      |
-------------------------------------+-------------------------------------
Changes (by jelte):

 * owner:  jelte => jinmei


Comment:

 Just wondering, have you considered waiting for a bit and trying again if
 get_remote_config fails (obviously with a real propagated error if it
 still doesn't work after X attempts)?

 Since the module should have already announced its own existence there
 should be no deadlock there, and while forcing its own restart may work,
 it might stop working if we implement automatic unregistering modules that
 die (unless we get to implementing config/remote config for non-running
 modules first).

 The code itself seems ok, just a few comment nits:

 ddns.py.in:
 - typo on line 202 (propagte)
 - line 318: '# class is optional per spec.' I guess that should say 'not
 optional' which would make the rest of the comment make more sense too :)

 For the rest it looks good

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