[2435] Make ZoneUpdater return a corresponding RRsetCollection for itself

[2435] Throw FindError in case there is a DataSourceError in ZoneFinder

[2435] Don't throw in case of out of zone queries

[2435] Handle find() with RRClass that may not be found

[2435] Test RRsetCollection against both MockAccessor and TestSQLite3Accessor

[2435] Remove the DatasrcIter implementation for now

It is not complete and unused, as the getBeginning() and getEnd()
methods simply throw now.

[2435] Describe that a reference to the updater is kept

[2435] Rename DsIter iterator class to DatasrcIter

[2435] Fix copyright year of files

[2435] Implement datasrc version of RRsetCollection.

[2432] Fix more cases of \return (use singular form).

[2432] Use prefix operator++.

[2432] Minimize code.

[2432] Use ASSERT_TRUE to not crash test suite in some places.

[2432] Fix use of \return (use singular form).

[2432] Use \throws in API doc.

[2432] Remove explicit template instantiation in one case.

I thought that explicit type name made the code more readable, but maybe
that's not the case so I've removed it now.

It still needs to be specified in the other constructor.

[2500] added notes that MNAME/RNAME must not be quoted string.

[2500] extracted createName() from soa.cc so it can be commonly used.

[2317] Check for duplicated option definitions.

[2317] Added remaining unit tests for DHCP option definitions parser.

[2317] Test configuration of option definition comprising a record.

[2317] Parse record-types field in DHCPv4 option definition configuration.

[2317] Store configured option definitions in the CfgMgr object.

[2320] Minor memory leak in Dhcpv4_srv test fixed.

[2320] dynamic_cast in Subnet::getPool() removed

[2435] Make ZoneUpdater return a corresponding RRsetCollection for itself

[2317] Added generic container holding option spaces.

[2317] Create V4 option definition in the definition parser.

[2320] Minor changes to the documentation

[2436] Include some headers explicitly

It worked without this, this is just to make sure it doesn't break.

[2317] More strict checks for option name format.

[2436] Use example.org instead of root zone

Just to be closer to real life, no special reason.

[2436] Test that warnings don't disrupt loading

[2436] More renaming from validation to check

For consistency.

[2436] Fix comment typo and make it clearer

[2436] Proper terminology in comment

[2436] Tweak log messages

And their IDs. Just slight clarifications.

[2436] Remove fixed bug from man page

[master] Added ChangeLog entry for #2313.

[master] Merge branch 'trac2313'

Conflicts:
src/lib/dhcpsrv/cfgmgr.cc
src/lib/dhcpsrv/cfgmgr.h

[2435] Throw FindError in case there is a DataSourceError in ZoneFinder

[2435] Don't throw in case of out of zone queries

[2435] Handle find() with RRClass that may not be found

[2435] Test RRsetCollection against both MockAccessor and TestSQLite3Accessor

[2435] Remove the DatasrcIter implementation for now

It is not complete and unused, as the getBeginning() and getEnd()
methods simply throw now.

[2435] Describe that a reference to the updater is kept

[2435] Rename DsIter iterator class to DatasrcIter

[2435] Fix copyright year of files

[2436] style fixes: naming convention; position of &; combine short lines

[2500] added unit tests for SOA::compare().

totally irrelevant to the subject of the task, but since there was
no test for this method before, I'm trying to make it a little better
at this opportunity.

[2500] corrected some test data, making names in SOA RDATA absolute.

they were accidentally (incorrectly) accepted due to the lenience of
string version of Name constructor but should have been rejected (and
fixed).

[2500] re-enabled non-absolute name in SOA Rdata for loadzone tests.

this will pass on completion of this task.

[2500] added some more doc and more tests

[2500] check the case SOA-ish text begins with a new line (should be rejected)

[2500] use MasterLexer for "from string" ctor's of SOA rdata

[2317] Created common function to get configured parameter from a storage.

[2436] explicitly include <string>, just in case.

[2436] constify

[master] Merge branch 'trac2437'

[2317] Added parser classes for option definitions.

[2320] Changes after review.

[master] Updated ChangeLog entry for #2315.

The updated description specifies that the change was in the DHCP code.
Also, the updated entry points out that new code is used internally by
the DHCP server. Okayed on jabber.

[2317] Added option-def section in dhcp4.spec.

[2313] Trivial: updates to some comments as a result of the second review.

[2436] Adjust old loadzone tests

They were expecting the loadzone would just warn, not fail, on the bad
zone data.

[2313] Changes as a result of the review.

[2436] Remove the validation from the b10-loadzone

It is now done in the ZoneLoader in the C++ backend, so don't duplicate
the code.

[2436] Update the mock finder, so it works with zone validation

[2436] Perform the validation in ZoneLoader

TODO: Modify this commit and use the datasource RRsetCollection. This
way it doesn't work.

[master] Added ChangeLog entry for #2315.

[2436] Test there's validation in ZoneLoader

[2436] Document validation in ZoneLoader

It should throw an exception if the validation fails.

Merge remote-tracking branch 'origin/trac2435' into master

[master] Merge branch 'trac2315'

Conflicts:
src/bin/dhcp4/config_parser.cc
src/bin/dhcp6/config_parser.cc

[2320] Changes after review, part 1

[2320] Pkt4::setType() is now implemented properly.

[master] Added ChangeLog entry for #2545.

[master] Merge branch 'trac2545'

Conflicts:
src/lib/dhcpsrv/Makefile.am

[2545] Corrected a typo in dhcpX.spec files.

[2225_statistics] reload the counters module before starting tests

[2225_statistics] add multi-module tests for the scenarios

The following two scenarios are intended here:

  - The counters are incremented in the scenario, which notify_out is solely
    invoked and default statistics spec is used.

  - The counters are incremented in the scenario, which XfroutServer is invoked
    and the same spec file is loaded in several times

[2225_statistics] correct the module name according to renaming to counters.py

[2225_statistics] simplify too detailed documentation in start_timer() and stop_timer()

[2225_statistics] correct documentation of inc() and dec()

[2225_statistics] add a common helper function for incrementing or decrementing a counter

also inc() and dec() share its implementation.

[2225_statistics] add note to _concat()

[2225_statistics] rename counter.py to counters.py and make the related changes for the review comment

[2225_statistics] Make various documentation updates

These fix typos, grammar and also rewrite some comments so that they are
more clear to the reader.

[2225_statistics] rename dump_statistics() to get_statistics()

It's due to the review comment.

[2225_statistics] update document as it is

It's due to the review comment.

[2225_statistics] add helpful comments to the complicated expression

Also rsplit() is used instead of split() to be simplified.

It's due to the review comment.

[2225_statistics] rename to clear_all()

It's due to the review comment.

[2225_statistics] rename start()/stop() to start_timer()/stop_timer()

Due to the review comment.

[2225_statistics] remove 'else' statement

In stead of being removed, it just returns if DataNotFoundError is raised.

Due to the review comment.

[2225_statistics] update document of Counters.stop()

explicitly distinguishing duration time from date time.

Due to the review comment.

[2225_statistics] use timedelta.total_seconds() instead of redundant computation

Due to the review comment.

[2225_statistics] remove an unnecessary formating

Due to the review comment.

[2225_statistics] remove an unnecessary try/expect statement

Due to the review comment.

[2225_statistics] add step as kwarg into inc() and use it in dec() with step=-1

Due to the review comment.