BIND 10 #1414: Zonemgr/secondary_zones[]/class required

[BIND 10 Development]

#1414: Zonemgr/secondary_zones[]/class required
-------------------------------------+-------------------------------------
                   Reporter:         |                 Owner:  UnAssigned
  jinmei                             |                Status:  reviewing
                       Type:         |             Milestone:
  defect                             |  Sprint-20111220
                   Priority:  major  |            Resolution:
                  Component:         |             Sensitive:  0
  configuration                      |           Sub-Project:  Core
                   Keywords:         |  Estimated Difficulty:  4
            Defect Severity:  N/A    |           Total Hours:  2
Feature Depending on Ticket:  none   |
        Add Hours to Ticket:  0      |
                  Internal?:  0      |
-------------------------------------+-------------------------------------
Changes (by jelte):

 * owner:  jelte => UnAssigned
 * status:  assigned => reviewing
 * totalhours:  0 => 2


Comment:

 the method that handles the configuration update needs access to the
 ccsession (for the get_default_value() so we don't have to hardcode a
 value here).

 Ideally, we should change the basic 'new_config' object we pass around
 (globally), so that is has the value, and its spec (including its default
 value). However, that would be way too big a change for this ticket. So
 for now I've only added the moduleccsession to the update config method
 (and added a minimal fake one in the tests), and left the rest of the code
 as much intact as possible.

 Oh I've also improved the error if no zone name was given (i.e. if you do
 config add secondary_zones; config commit). It was a similar confusing
 error caused by trying to directly access a nonexistent key in a dict).

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