BIND 10 #2421: don't reject configuring entire inmemory datasrc due to one broken zone

[BIND 10 Development]

#2421: don't reject configuring entire inmemory datasrc due to one broken zone
-------------------------------------+-------------------------------------
                   Reporter:         |                 Owner:  muks
  jinmei                             |                Status:  closed
                       Type:         |             Milestone:
  defect                             |  Sprint-20121120
                   Priority:         |            Resolution:  fixed
  medium                             |             Sensitive:  0
                  Component:  data   |           Sub-Project:  DNS
  source                             |  Estimated Difficulty:  4
                   Keywords:         |           Total Hours:  2.49
            Defect Severity:  High   |
Feature Depending on Ticket:         |
        Add Hours to Ticket:  0      |
                  Internal?:  0      |
-------------------------------------+-------------------------------------
Changes (by muks):

 * status:  reviewing => closed
 * resolution:   => fixed


Comment:

 Merged to `master` branch in commit
 `bbb0752c34f7712864b596f208cefd9746eab287`:

 {{{
 * e650a0f [2421] Put code inside EXPECT_NO_THROW
 * 744d1be [2421] Update comments about test data
 * b37e972 [2421] Include zone name in log messages
 * 7498576 [2421] Update DATASRC_LOAD_FROM_ITERATOR_ERROR description
 * 49d6a08 [2421] Add unit tests for broken zones during configure()
 * 4bc7823 [2421] Check that the load error messages are logged
 * 2332578 [2421] Add a lettuce testcase where the zone file doesn't exist
 * bc457a1 [2421] Throw ZoneLoaderException even for MasterLoadError
 * 0d08fcb [2421] Catch ZoneValidationException at a higher level
 * 5ba2cab [2421] Update InMemoryClient doc about exceptions
 * 9e1942f [2421] Throw AddError instead of OutOfZone when loading out-of-
 zone data
 * 6c30834 [2421] Add zone exceptions hierarchy
 * cec4bf4 [2421] Handle exceptions when loading a zone
 * 80c2a36 [2421] Add a testcase for empty zone too (which should throw
 EmptyZone)
 * ecace4b [2421] Change test data to throw ZoneDataUpdater::AddError
 * 0ff35f6 [2421] Add lettuce test with broken zone
 }}}

 Resolving as fixed. Thank you for the reviews Michal and Jinmei.

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