BIND 10 #2113: Configuration plugin check for datasources

[BIND 10 Development]

#2113: Configuration plugin check for datasources
-------------------------------------+-------------------------------------
                   Reporter:         |                 Owner:  vorner
  vorner                             |                Status:  reviewing
                       Type:         |             Milestone:
  defect                             |  Sprint-20120731
                   Priority:         |            Resolution:
  medium                             |             Sensitive:  0
                  Component:         |           Sub-Project:  DNS
  configuration                      |  Estimated Difficulty:  5
                   Keywords:         |           Total Hours:  0
            Defect Severity:  N/A    |
Feature Depending on Ticket:         |
        Add Hours to Ticket:  0      |
                  Internal?:  0      |
-------------------------------------+-------------------------------------
Changes (by jelte):

 * owner:  jelte => vorner


Comment:

 The code looks ok, just a few comments:

 I think the error string in case of bad classes could be better; it prints
 "Unrecognized RR parameter string" which isn't really helpful IMO.

 I also found a small problem with the 'cache disabled' warning I made you
 add in a previous review; I added a MasterFiles configuration with my zone
 and loaded it, but the ClientList logs a warning:
 2012-07-31 11:30:46.507 WARN  [b10-cfgmgr.datasrc] DATASRC_LIST_NOT_CACHED
 zone tjeb.nl/IN not cached, cache disabled globally. Will not be
 available.

 However, the zone appears to be loaded and served fine.

 {{{
 2012-07-31 11:35:39.506 WARN  [b10-cfgmgr.datasrc] DATASRC_LIST_NOT_CACHED
 zone tjeb.nl/IN not cached, cache disabled globally. Will not be
 available.
 }}}
 Looks like that is because it isn't called with allow_cache=True in
 datasrc_config_plugin.

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