[2349] Documentation for the pimpling of two classes

[2223] completely pull out sigs and check entire set

[2342] Separate out lease manager creation into separate factory functions

[2207] Implementation of the getZoneReloader

The concrete version for local table segment and reloader.

[2207] Docs and capitalization fix

[2207] Tests for the getZoneReloader

[2207] interface of the getZoneReloader

[2198] Update comment

[2198] Test the non-blocking variant of Mutex::Locker

[2198] Check pthread_mutex_trylock()'s return value

[2198] Revert include to local

[2198] Stop using direct lock methods on Mutex inside InterprocessSyncFile

[2198] Make direct methods on Mutex private

[2198] Add a block argument to Mutex::Locker constructor

[2198] Add AlreadyLocked exception to Mutex::Locker

Merge branch 'master' into trac2302

[2268] Make minor code cleanup

[2168] Untabify

[2268] Remove per-test iterator classes and use MockVectorIterator instead

[2268] Add MockVectorIterator class

[2207] Deprecate InstallAction

As we want to directly modify the ZoneTable in ZoneReloaderLocal
(because it is memory-type dependant), we don't need the InstallAction
for anything. If we don't need it, we don't pass it as constructor
parameter -- it'll be called from a factory anyway and the factory may
know we don't need it.

Also adding a setTable() method to ZoneTableHeader. The header had no
way to specify the table currently and we need it for the tests. The
method is currently tentative, we'll probably replace it by something.

[2207] Check the load action returns something

NULL makes no sense, so throw in that case.

[2207] Change signature of LoadAction

We allocate the ZoneData in the LoadAction, to match what happens in
ticket #2268 -- there's a function to load zone which allocates it
itself, we want to use it.

[2207] Rename updater to reloader

This might be less confusing.

[2205] added DataSrcClientsBuilder class, main part of configurator thread

[2205] Merge branch 'trac2332' into trac2205

[2332] missing cleanup: removed an unused function.

[2205] Merge branch 'trac2332' into trac2205

[1858] (unrelated cleanup) removed BIND10_START_AS_NON_ROOT_AUTH log.

with the introduction of socket creation this log message hasn't been
making sense.  It would be rather confusing and harmful.

[2268] minor editorial cleanups

[2349] Same for RunningQuery

See 26483ea27726d4c1097485196a0908c565478188

[2270] Comment about code overlap added.

[2270] Documentation for DHCPv4 config. parser written.

[2349] clean up ForwardQuery objects in tests

an attempt to clean up memory; ForwardQuery objects are supposed to delete themselves when they are no longer needed, but due to the direct calls in tests this often does not happen.

To keep internals in recursive_query to a minimum, I did add something to the API; ForwardQuery is now a public class, a pointer of which is returned by RecursiveQuery::forward. Tests that mess around can then delete the query themselves, and the way these are handled does not have to be changed.

Ideally, we should however not need unscoped allocated memory in the first place; we either need to use a centrally managed pool of objects to (re)use, or we need to change the entire design that needs these objects in the first place.

However, these are both non-trivial, and probably better suited in the main resolver work that has been planned.

[2270] DHCPv6 pool parser fix.

[2270] DHCPv4 parser fixes. Tests now pass.

[2304] Minor additions in comments.

[2155] added some notes to the mapping tables of counter item

[2304] Fixes for the code review.

Includes adding missing tests for signed integer values

[2155] update test case according to the review comment

[2268] Remove obsolete comment (the bug was fixed)

[2268] Move FileNameDeleter out of InMemoryClient

[2268] Update API doc

[2268] Move ZoneDataLoader into an unnamed namespace

[2268] Move ZoneDataLoader class out of its header

[2268] Use loadZoneData() inside InMemoryClient

[2268] Implement loadZoneData() functions

[2268] Move ZoneDataLoader to a separate file

[2268] Rename load method to loadInternal()

[2268] Add a comment about InMemoryClient construction after setThrowCount()

[2268] Add any attached RRSIGs when loading from an iterator

For example, the MemoryIterator returns TreeNodeRRsets which may have
attached RRSIGs.

This also fixes the failing unittest
MemoryClientTest.loadRRSIGsRdataMixedCoveredTypes.

[1858] cleanup: reorder log messages

[1858] stop sending hopeless signal when it once fails with EPERM.

[1858] added a test case where killing child process results in exception.

[1858] rename terminate_children() kill_... for consistency

[1858] cleanup: removed unnecessary list() conversion from dict values.

dict.values() is an iterable, so we can use it directly in the for statement.
(and we don't modify components within the loop).

[1858] refactoring: unify the SIGTERM/KILL cases into a single helper method.

[1858] explicitly set close_fds to True in sockcreator's Popen().

[2207] Add missing test file

Someone O:-) forgot to add it in the ^^ commit.

[2207] Implementation of the zone updater

The interface was slightly updated, to suit the needs (added some more
parameters). Some of it might be wrong, but I'm not sure about it.

[master] add changelog entry for #2340

Merge branch 'trac2340' of ssh://git.bind10.isc.org/var/bind10/git/bind10 into trac2340

with fixing Conflicts:

Merge branch 'master' into trac2340
with fixing conflicts in
ChangeLog
examples/README

[2207] Tests for the zone updater

Doesn't compile, there's no (not even empty) implementation of many
methods of the updater.

[2270] Configuration parser for DHCPv4

[2340] removed a bogus 'test' within a test(1) expression.

[master] changelog for #2339

[2339] changed the search order of python executable: seek python3.x first.

this works around a bit awkward installation setup where there's a link
from "python3" to "python3.x" but not from "python3-config" to the
corresponding "python3.x-config".  That happens for recent versions of
Homebrew.

Compilation fix: usual static_cast<X*>(NULL)

Sunstudio is unhappy with plain NULL inside EXPECT_EQ and thinks it is
int. This makes it think otherwise.

Remove extra comma at the end of enum

This should fix compilation issue on sunstudio.

[master] added changelog for #2244

[master] a minor grammar fix to changelog entry.

[2244] missing update: Component.running() was renamed is_running().

[2244] renamed BaseComponent.is_failed() to is_restarting().

so it sounds more natural as an English term.

[2244] fixed a typo in a log message.

[2244] rename Componet.running() is_running() for consistency.

this branch introduced is_failed() (and failed() was already defined for
a different a purpose), so for consistency it would be better to name
the running version is_xxx too.

[2244] cleanup: reordered log message file

[2244] make sure restarting components only when they are in config.

this should fix the main problem of this ticket.

[2244] a minor editorial fix to the previous comment update

[2244] added comment about has_comonent as it may not be very obvious.

[2244] added a simple has_component() method to configurtor.

should be pretty trivial, so added a couple of test cases as part of
other tests.

[2244] make sure configurtor's build_plan() handles 'failed' component.

this fixes the test case introduced at the beginning of the branch.

[2244] introduced a new "Failed" state of Component.

[2244] added a test case that reproduces one aspect of the problem.

the configurator should be able to delete a component to be removed
even if the it's failing (unexpectedly).  The current implementation
doesn't handle that condition correctly.

[2204] removed an obsolete comment that only made sense for the previous code.

[2204] added notes about future extensions of incremental datasrc config update

[2204] grammar fix in documentation

[2204] updated test comments without using "rollback".

as the new code actually doesn't do rollback operation internally,
even though the observable effect is the same.

[2204] updated comments about releasing old data source client lists

the previous one was not really accurate and was confusing.

[2204] cleanup: removed unused func, and renamed "ClientList" with "DataSrc".

getClientListClasses() was removed.  It's not used anymore.
a few methods named with "ClientList" were renamed with adding "DataSrc".
The resulting names are a bit long, but since there's no namespace
qualification, simply saying "client" can be confusing (it could be
interpreted to mean DNS clients, for example).

[2204] regression fix: make sure the datasrc config is installed to server.

[2204] removed server function/template param from configureDataSourceGeneric.

This is pure cleanup.  This function already doesn't use the server.

[2204] extracted swapDataSrcClientLists() from configureDataSource().

now configureDataSource() can take time.

[2204] completely replaced setClientList with swapDataSrcClientLists.

the test cases using setClientList were updated so they use
swapDataSrcClientLists (some of them work as a test for the "swap" itself).
now we don't need setClientList, so it was removed.

[2204] simplify configureDataSource by always creating a new lists and swap.

so we don't have to worry about what are in the current lists or rollback
operations.
swapDataSrcClientLists() is newly introduced for AuthSrv.  No direc tests
yet (technically bad in terms TDD but the definition is very simple), which
will be provided in the next step.
the lock is now moved inside swapDataSrcClientLists().

note: even though this version builds everything, the amount of work
should be mostly the same because the only save is to create the empty
ClientList when the new and old have the same class of client.  The expensive
part is ClientList::configure().  This version doesn't need any more call
to configure() than the old version.

[2204] changed internal representation of auth_srv's "client_lists_" to ptrs.

this will be necessary later in this branch.
also renamed the member variable "datasrc_client_lists_" as the mere "client"
can be ambiguous.

[2292] Add a TODO about re-adding tests

Planned for merge #2268

[2292] Clarify test

And make it slightly stronger (it now checks something that should be
mostly obvious too, but who knows, with software).

[2292] Pass the old zone back

Instead of releasing it directly. While the internal release was more
convenient, it didn't allow for swapping things fast under a mutex and
then spending the time releasing it unlocked.

[2292] Don't pass SegmentObjectHolder

It is supposed to be mostly private. Use it internally only.