[2179] added a proposed Changelog entry

[2179] Supported a identifier in isc.cc.data e.g. "xxx/yyy/zzz[i]"

If a target module returns data whose key name is such a identifier type, the
stats module can identify statistics data. The stats module checks the
validation of data even if such a identifier is specified. But when the stats
shows statistics data e.g. via bindctl, this type of identifier is not shown
for now.

[2179] Supported a named_set data type

A named_set data type is supported in a spec file in each module for
statistics.

[2179] Supported differential statistics updates of Stats

Stats updates corresponding statistics data which each module returns.  For
that, the method "update_statistics_data()" was updated to merge recursively
old value and new value each other. As for this merging, a new method
"merge_oldnew" was introduced into stats.py. This method is intended to be
internally used so far.

More fix not to change commands' output to config channel.

[2136] moved invoking update_modules() to outside of the method

When the set command was used for collecting statistics data, the action was
randomly. But in the case that periodical polling is used for collecting
statistics data, update_modules doesn't need to be invoked so many times every
time update_statistics_data is invoked. Invoking update_module() causes an IO
wait for receiving statistics spec from Cfgmgr.

[2136] corrected the improper test

Not self.stats.mccs.specification but self.stats.modules['Stats'] should be set
the invalid spec_module into. And it should check whether StatsError is raised
by setting the owner to the self module name in the argument of command_show.

[2136] removed the do_polling method from the __init__ method of stats.Stats()

Because do_polling() is done when the show command is invoked, the do_polling
method didn't need to be done when the stats is starting.  Related to this
change, the b10-stats_test.py was changed. do_polling() is inserted before the
assertion of statistics_data.

[2136] added a note about a value from the Boss command show_processes

[2136] removed an unnecessary loop

[2136] revised the internal method _accum()

- Other conditions are considered. One case is that two args are string type,
  another case is that either of them is None type.

- _accum() was moved to outside of the Stats class for testing it individually.

- revised the FIXME description in update_statistics_data() according to the
  above changes

[2100] (not directly related) cleanup: always destroy old data in setData().

actually the previous implementation seems buggy; it returns the given pionter,
not the old data.  Worse, this code path isn't tested.  It's a typical example
of YAGNI; if we don't see the need for it, don't do it; if you cannot regist
the temptation of making it smarter, at least you should write tests.

At least ZoneTable doesn't need it, so I chose to clean it up.  If and when
we find it necessary, we should implementation from the scratch, and
test-driven.

[2100] use offset pointer for domaintree data.

not absolutely necessary for this branch, but that's the intent anyway.

[2100] use offset pointer for the underlying tree

[2100] cleanup: removed removeZone().

right now it doesn't work, and when we need to implement it, the internal
will need to be substantially updated anyway, so it doesn't make much sense
to keep the nonworkable version.

[2100] documentation update

[2100] updated the return type of findZone().

It now simply returns ZoneData as the data associated with the code;
the corresponding client implementation will create a zone finder from it.
also updated some doc.

[2100] return the created zone data from addZone.

[2100] improved exception safety.

[2100] revised addZone() iface: now it only takes name, creates data in it.

also completed the zone table deleter.

[2172] refactor sysinfo tests

[2100] ensure exception safety in ZoneTable::create using a holder class.

instead of using the ugly try-(catch-all)catch.

[2172] fix osx sysinfo tests for previous change

[2172] fix page_size regex

[2172] rearrange freebsd/osx hierarchy

[2096] Test decoding

Provide three more decoders of the data, using the RdataReader. This
does not link, as the reader has no bodies of the methods.

[1959] Added multicast option on the socket if remote addr is multicast.

[1959] Added generation of DUID prefix if it is not given by user.

Merge branch 'master' into trac2171

[2164] Add RRset::getSIGRdataCount()

[master] Add tests for RRSIG related parts of RRset class

[master] Untabify code

[2096] Code reorganisation

The foreach functions to manually iterate over the data are used in
tests only, so moved to the tests.

The specifications are used by the tests and encoding and will be used
by the decoding machinery too, probably, so we are making them
accessible and reside in a separate file.

[2096] (Unrelated) MessageRenderer isn't copyable

Discovered the hard way - it can crash.

Small trick with just-in-time initialization of a pointer in benchmark
is used instead of copying it and relying on the fact that it can be
copied while it was not yet used.

[2096] Test refactor: Make it type-parametric

This will help checking the various supported interfaces to decoding.

[2096] Move the test file

This is logically part of the next commit. But then, the git diff would
show the old file disappear and the new file appear with new content,
which would be hard to review, so it is split. This commit should
contain no changes in the content (hopefully), so can be skipped during
review.

[2096] Fix stupid naming error

When changing the names, this kind of got forgotten.

[master] Add logger_lockfile to .gitignore

[2136] Changes regarding the test config file

- moved b10-config_test.db from src/bin/stats/tests to
  src/bin/stats/tests/testdata

- renamed its name to b10-config.db

- changed its format to multiline

These changes are due to the reviewer's comment.

[2136] Update comment text

[2136] Minor whitespace update

[2136] Remove redundant tests

[2136] Indent code

[2136] Assign and use intval earlier

[2136] Add back part of a deleted comment

[2136] Make minor text updates

[2136] Re-indent code

[2136] Make some minor changes to poll-interval's description

[2136] Updated the do_polling method

A list of instances for stats to request is obtained from the show_processes
command of Boss instead of asking the components config of Boss.

[2136] Updated the "show_processes" to provide address of each module

This information is used for other module e.g. stats to count running instances
of same module. The address information is seen in the third index of the
array.

[2136] Introduced an address() method for the Component classes

It returns the "_address" attribute of the classes. This is used for the
"show_processes" command of Boss to provide address information.

[2136] Updated the 'show' command handler

The stats module decides if polling should be done by the the last time of
polling. If more than one seconds past since the last request to each module,
the stats module requests each module statistics data and then shows the latest
result. Otherwise, the stats module just shows statistics data which it has.

[2136] Separated the updating process from the receiving process

In the updating process, it communicates with the cfgmgr module for updating
statistics specification of each module. This fix is for separating such mixed
communication process.

[2136] Revised the test scripts and the config db file

- Fixed the incorrect previous fix.
  There should not be the inconsistency on number of mock Auth instances on
  both test scripts: b10-stats_test.py and b10-stats-httpd_test.py. In the
  previous change, there was the inconsistency in MockBoss class. Actually in
  b10-stats-httpd_test.py, the number of auth instances which Cfgmgr answers
  was four, but the number of actually invoked auth instances was one.

- Reduce the number of invoked mock Auth instances
  In theses test scripts, multiple Auth instances need to be invoked. But if
  four instances are actually invoked in these scripts, open files(ulimit -n)
  exceeds 256. In the environment where ulimit -n was set to 256, the
  b10-stats-httpd_test.py test failed actually.  In that case,
  b10-stats-httpd_test.py needs open files more than 256.  So the number of the
  auth instances in mock modules was reduced into 2 in the scripts. This fix is
  for the comment at #2136.

[2136] Fixed unfinished MockAuth objects

The extra MockAuth objects didn't shut down in
BaseModules.shutdown(). But these weren't needed for unittest of stats
httpd. So setting up and shutting up of them were moved to
b10-stats_test.py from test_utils.py.

[2136] some small suggested wording fixes to doc.

[2136] Fixed wrong checking: The variable key was assumed to be constant.

[2136] Removed 'trees' argument from 'getstats' command

[2136] Refactoring unittest and update to evaluate the next_polltime value as a
integer

[2136] Updated not to calculate next timing based on current timestamp in every
polling

[2136] moved creation of a stats.Stats object in each test case

[2136] kept consistency in unittest due to the previous changes

[2136] do group_sendmsg too all modules then do group_recvmsg in
do_polling for efficiency

Also added setting timeout value to one second for precise and
refactoring timeout handling

[2136] added a new message id for getting invalid statistics data from the
module

[2136] updated bind10-guide.* due to removing set command from stats module

[2136] added some misc changes

Changes in b10-stats_test.py:
 - added deeply checking statistics values of each of module and
   instance

 - added stats_server.shutdown() at the end of the test_config method.
   Because the running stats_server was never shut down even after
   stats_server.run() was invoked.

Changes in test_utils.py:
 - Changed the way of invoking multiple auth instances just for efficiency

[2136] renamed 'pid' to 'mid' and change the default value -1 to None on
the second argument of the the 'update_statistics_data' method

Instead of PID(Process Id) used in previous version of stats codes,
'mid' (module id) is used for identifying multiple instances of same
module. 'lname' of CC-session in stats codes is assigned to it.

[2136] removed the obsoleted command 'sendstats' from the mock modules

[2136] removed 'set' command and introduced a 'poll-interval' config item

The description about 'set' command was removed from the manpages, and
A 'poll-interval' config item was introduced into the manpages. Also
the implement, the testcases and specification related to the set
command were removed from source codes and the specfile.

[master] Merge branch 'trac1958'

Conflicts:
tests/tools/perfdhcp/Makefile.am

Statistics data transfer method is changed
 from peliodic sending via statistics channel
 to passive retrieval via config session

- Added "getstats" command to retrieve statistics data via config session.
  = type "Auth getstats" command using bindctl shows statistics data.
- Removed StatisticsIntervalTimer
- Removed "sendstats" command
- Changed that each config command can return values.
  - returns Element::createlist(), Element::fromJSON("{}") or Statistics

Opinion:
- Each command should return conditions and values using createAnswer()
   rather than using throw/catch.

[2100] simplify: move definitions to the main class instead of pimpl.

this is mostly private class and is not expected to be referenced outside
of this psuedo module ("memory"), so it's raltively safer.
also, the ZoneTable structure will be embedable in a shared memory space,
so introducing the another pointer indirection will make it unnecessarily
complicated.

still don't pass tests.

[2100] make the new implementation compile.

added a mostly-empty implementation of ZoneData.
at this point it's nothing more than "buildable" code.  tests still don't pass.

[2100] copy the old zone table implementation and tests under "memory" dir.

right now it's a naive copy.  the test wouldn't even compile.

[2172] fixed a trivial typo

[1959] Enabled Solicit packets creation and send.

[1958] Implemented changes suggested in the second code review.

[2096] Help for the RDataReader interface

Including the expected usage scenarios.

Also, the interface was slightly refined - renaming of things, adding
support for the RRSig iteration.

[1959] Add some more comments to the test_control.cc

[1959] Enable broadcast addresses for perfdhcp's DHCPv4 mode.

[1959] Enable use of broadcast remote address.

[2172] Add OSX support to isc-sysinfo

Mostly the same as FreeBSD, except for swap usage

[2096] Interface of the RDataReader

The oposite of the isc::datasrc::memory::RDataEncoder.

Only the methods are drafted, no documentation yet and no member
variables listed.

[1959] Set packet v4 defaults correctly

[2066] docs: Update links to the ACL chapter

Instead of talking the same over and over again, we now link the common
ACL chapter.

Merge #2133

[2133] Minor fix

The previous version was slightly misleading, the reader could
understand it the queries would be just slow as answered from the DB,
instead of not answered at all.

[2066] docs/ACLs: Some examples and how to bindctl it

[2133] a few small changes in the new documentation

[master] update changelog for merge of 2126

since the b10-showtech had not been released yet, I just replaced its name in the existing changelog entry

[master] Merge branch 'trac2126'

[1959] Receive packets from the server.

[master] Update .gitignore files

[2171] Set g+s,g+w permissions on localstatedir

This is usually $prefix/var/bind10-devel/. We set mode to 2770 oct.

[2171] Remove redundant mkdir (already exists in cfgmgr)

Besides, loadzone doesn't do anything with this directory.

[2066] Explain ACLs

Not complete. Split done at the end of the day, without too much logic
at which point.

[master] changelog for #2147

[master] Merge branch 'trac2147'

[1959] Documentation cleanup.

Merge branch 'master' into trac2105