[master] ChangeLog for trac3259

[master] Merge branch 'trac3259'

[master] Merge branch 'master' of ssh://git.kea.isc.org/git/kea

[master] Trivial comment fix

[master] Added ChangeLog entry 1046 for #3601

[master] Removed periods from log messages

    src/lib/dhcpsrv/dhcpsrv_messages.mes

[master] MemfileLeaseMgr now supports upgrading/downgrading lease files

    Merged in branch 'trac3601'

[3601] More review comments

Minor log message and comment wording

[master] Added ChangeLog entry 1045 for #4095

[master] Added client class storgae classes to libdhcpsrv

    Merged in #4095.

[master] Added a check on rcode avoiding unused errors

[4095] reindent

[4095] spelling & co

[4095] spelling

[3601] Addressed additional review comments

doc/guide/admin.xml
src/lib/dhcpsrv/dhcpsrv_messages.mes
src/lib/dhcpsrv/lease_file_loader.h
    minor clean up, typos

src/lib/dhcpsrv/tests/csv_lease_file4_unittest.cc
src/lib/dhcpsrv/tests/csv_lease_file6_unittest.cc
    removed unnecessary use of scoped_ptr

src/lib/util/csv_file.cc
    CSVRow::trim() - replaced use of std::vector<>::erase with resise

src/lib/util/versioned_csv_file.h
src/lib/util/versioned_csv_file.cc
    VersionedCSVFile::next() - reorganized to use input_schema_state_
    VersionedCSVFile::columnCountError() - new convenience method
    minor cleanups

[master] Updated git hash (forgotten)

[master] Merge remote-tracking branch 'github/master'

Merged in change made to GitHub for ticket trac4111.

[master] Finished merge of trac4060a (Kea multi-threaded)

[master] Merged trac4060a (Kea multi-threaded)

[4095] Implemented new classes for storing client class definitions

src/lib/dhcpsrv/client_class_def.cc
src/lib/dhcpsrv/client_class_def.h
src/lib/dhcpsrv/tests/client_class_def_unittest.cc
    New files that define client class storage classes and tests

src/lib/dhcp/classify.h
    Amended commentary to reflect how these classes are now being used.

src/lib/dhcp/option.h
    Added typedefs:
        boost::shared_ptr<Option> OptionPtr;
        boost::shared_ptr<OptionCollection> OptionCollectionPtr;

src/lib/dhcpsrv/Makefile.am
    Added entries for new files

src/lib/dhcpsrv/tests/Makefile.am
    Added entries for new file

src/lib/eval/token.h b/src/lib/eval/token.h
    Added typedef:
        boost::shared_ptr<Expression> ExpressionPtr;

[3601] Addressed review comments, added ability to downgrade

Several minor cleanup items based on review comments.  Implemented
support for downgrading files from newer schema versions:

doc/guide/admin.xml
    amended text on upgrading memfile to discuss downgrading

src/lib/dhcpsrv/dhcpsrv_messages.mes
src/lib/dhcpsrv/lease_file_loader.h
    revamped log messages to accomodate downgrading

src/lib/dhcpsrv/memfile_lease_mgr.cc
src/lib/dhcpsrv/memfile_lease_mgr.h
    added commentary to MemfileLeaseMgr ctor
    automatic conversion logic accomdates both upgrading and downgrading

src/lib/dhcpsrv/tests/csv_lease_file4_unittest.cc
src/lib/dhcpsrv/tests/csv_lease_file6_unittest.cc
    replaced tooManyHeaderColumns test with downGrade test

src/lib/util/csv_file.h
src/lib/util/csv_file.cc
    added CSVRow::trim()

src/lib/util/tests/csv_file_unittest.cc
    added CSVRow.trim test

src/lib/util/tests/versioned_csv_file_unittest.cc
    added VersionedCSVFileTest.currentSchemaTest test
    replaced tooManyHeaderColumns test with downGrading test
    revamped tests to check  getInputSchemaState() and needsConversion()

src/lib/util/versioned_csv_file.h
    Updated commentary to describe downgrade support

src/lib/util/versioned_csv_file.cc
    enum InputSchemaState
    input_schema_state_
    input_header_count_
    getInputHeaderCount()
    getInputSchemaState()
    needsConversion()

    next(CSVRow& row) - now supports downgrading rows
    validateHeder() -  now throws if called when no schema has been defined,
    and supports downgrading rows

[3259] Changes after review

1. Updated Kea guide to note that removing the hooks-libraries
   configuration element does not always have the expected effect.
2. Moved some documentation from the dhcp_parsers.cc file to the .h file.
3. Expanded checking of the contents of the hooks-libraries configuration
   element.

[master] Updated git hash

[master] Finished merge of trac4091 (hex strings)

[master] Merged trac4091 (hex strings)

[4091] Changed to require at least one hexdigit

[4091] Addressed comments (0x prefix, deciding in constructor)

[4091] Better odd check

[4091] Missed changes

[master] Updated git hash

[master] Finished merge of trac3927 (default in option-def)

[master] Merged trac3927 (default in option-def)

[3927] Fixed typo in previous change

[3259] Corrected a typo.

[3927] Addressed minor comments

[3601] Added discussion of memfile lease file upgrading to admin guide

[3601] Added unit tests for automatic LFC upgrade

src/lib/dhcpsrv/tests/memfile_lease_mgr_unittest.cc
    Added two new unit tests to verify lease file upgrade
    via automatic LFC invocation:
        TEST_F(MemfileLeaseMgrTest, leaseUpgrade4)
        TEST_F(MemfileLeaseMgrTest, leaseUpgrade6)

[3601] Memfile_LeaseMgr now runs LFC automatically if lease files need upgrading

Memfile_LeaseMgr loads the lease files during instantiation. If it detects
that any of the files loaded are out of date, it will invoke LFC directly,
whether or not LFC is enabled.

src/lib/dhcpsrv/dhcpsrv_messages.mes
    added two new log messages:
        DHCPRSV_MEMFILE_UPGRADING_LEASE_FILES
        DHCPSRV_MEMFILE_NEEDS_UPGRADING

src/lib/dhcpsrv/lease_file_loader.h
    LeaseFileLoader::load() - emits DHCPSRV_MEMFILE_NEEDS_UPGRADING log
    when an out of date file is detected

src/lib/dhcpsrv/memfile_lease_mgr.cc
    LFCSetup::setup() - added run_once_now parameter, which causes the
    method to invoke the LFC callback immediately, regardless of the value
    of LFC interval.
    Memfile_LeaseMgr::Memfile_LeaseMgr() - added logic to track if
    files loaded need upgrading and pass that into lfcSetup()

    Memfile_LeaseMgr::loadLeasesFromFiles() - change to return a boolean
    true if any of the files loaded need upgrading.

    Memfile_LeaseMgr::lfcSetup() - added upgrade_needed parameter, which
    is passed through to LFCSetup::setup() as "run_once_now"

[3601] Added methods to VersionedCSVFile for tracking when updating is needed

src/lib/util/csv_file.h
    Made recreate() virtual

src/lib/util/versioned_csv_file.h
src/lib/util/versioned_csv_file.cc
    Added several methods to VersionedCSVFile:

    getValidColumnCount() - returns number of valid columns in header

    recreate() - wraps base class method, ensuring valid column
    count gets set to number of defined columns for new files

    needsUpgrading() - returns bool true if file schema is out of date

    getInputSchemaVersion() - returns schema version found in file

    getSchemaVersion() - returns current schema version

    getVersionedColumn() - returns the column definition for a given index

src/lib/util/tests/versioned_csv_file_unittest.cc
    Added checks for new methods to existing tests

[4091] Implemented, need tests

[4091] Renamed main (cf #4114)

[4060a] Cleaned up (partially, i.e., keep ugly but working code)

Merge branch 'trac4090'  Add support for OptionSubstring to the eval code

[trac4090] Update per second set of review comments

Remove change log entry as it isn't needed

Update eval log description in user guide.

[trac4090] Add the forgotten eval_log.c and eval_log.h files

[trac4090] Setup logger for eval

[trac4090] Update per review comments.

Update per the review comments except for the logging
stuff which will be done next.

[3601] CSVLeaseFile4 and CSVLeaseFile6 now derive from  VersionedCSVFile

src/lib/dhcpsrv/csv_lease_file4.h
src/lib/dhcpsrv/csv_lease_file4.cc
    - CSVLeaseFile4 now derives from VersionedCSVFile
    - initializes schema metadata not just column names
    - uses VersionedCSVFile methods rather than CSVFile methods

src/lib/dhcpsrv/csv_lease_file6.h
src/lib/dhcpsrv/csv_lease_file6.cc
    - CSVLeaseFile6 now derives from VersionedCSVFile
    - initializes schema metadata not just column names
    - uses VersionedCSVFile methods rather than CSVFile methods

src/lib/dhcpsrv/tests/csv_lease_file4_unittest.cc
    - Added new unit tests:
    TEST_F(CSVLeaseFile4Test, mixedSchemaload)
    TEST_F(CSVLeaseFile4Test, tooFewHeaderColumns)
    TEST_F(CSVLeaseFile4Test, invalidHeaderColumn)
    TEST_F(CSVLeaseFile4Test, tooManyHeaderColumns)

src/lib/dhcpsrv/tests/csv_lease_file6_unittest.cc
    - Added new unit tests:
    TEST_F(CSVLeaseFile6Test, mixedSchemaLoad)
    TEST_F(CSVLeaseFile6Test, tooFewHeaderColumns)
    TEST_F(CSVLeaseFile6Test, invalidHeaderColumn)
    TEST_F(CSVLeaseFile6Test, tooManyHeaderColumns)

[3601] Added isc::util::VersionedCSVFile

src/lib/util/versioned_csv_file.h
src/lib/util/versioned_csv_file.cc
    New files which implement VersionedCSVFile, CSV file which can
    support mulitple schema versions

src/lib/util/tests/versioned_csv_file_unittest.h
src/lib/util/tests/versioned_csv_file_unittest.cc
    new files for Unit tests for VersionedCSVFile

src/lib/util/Makefile.am
    added new files

src/lib/util/csv_file.cc
    includes read error message if header fails to validate

src/lib/util/csv_file.h
    removed @todo for 3626, no longer applicable

src/lib/util/tests/Makefile.am
    added versioned_csv_file_unittest.cc

[master] Fixed 4o6subnetInterfaceId for not 64 bit machines (from hackathon94)

[master] Use LeaseMgrFactory::instance() rather than constructor.

This fixes issues reported by the coverity scans recently.

Merge branch 'trac4111'

[4111] constify

[4111] Added an emptyOptionData unit test

[4111] Style in checkEmpty

[3927] Added a defaultSpaceOptionDefTest unit test

[3927] Fixed spurious 4 in DHCPv6 guide

[4111] added a test case for an empty Option6AddrLst.

also renamed a local variable to coform to coding guideline (don't use
the camel style for variables)

[master] AUTHORS updated after recent changes.

[master] Merge branch 'trac4105' (4o6 configuration parameters)

[4105] Grammar fixed.

[4105] config parser now uses getOptionalParam() methods.

[4105] Changes after review:

 - getters/setters implemented in Cfg4o6.
 - extra space removed.

[master] Finished trac4027 aka Pkt4o6 merge

[master] Merge branch 'master' of ssh://git.kea.isc.org/git/kea

[master] Merge remote-tracking branch 'github/master'

[4027] removed an unused variable

(besides, this type of file-static definition is dangerious - it can lead to
static initialization fiasco).

[4027] added some explanatory comments for Pkt4o6::pack().

[4027] added some references

Merge branch 'trac4027' of github.com:isc-projects/kea into trac4027

with resolving conflict (the HEAD version wasn't complete)
src/lib/dhcp/tests/pkt4o6_unittest.cc

[4027] Minor corrections.

[4027] use a single underscore version of uint8_t for consistency

(and perhaps the double-underscore version is non-standard)

[master] Merged trac4107 (HCPv4-over-DHCPv6 message and option definitions)

[4107] Added several comments.

[4027] added some more comments for tests

[4027] added unit tests for Pkt4o6Test

(with other some small cleanups)

[4107] Minor comments update for options 100-102

[4105] Implement 4o6-interface-id parameter.

[4107] Added new standard options in the guide table

[4107] Added the 2 private options (now update the doc)

[4105] Subnet4 parser updated to accept 4o6-interface, 4o6-subnet parameters

[4107] Ported changes from fd4o6 (so not finished)

[4027] enable DHCP4o6 option macros

[fd4o6] Adjusted Pkt6 pointer type

[4027] incorporated part of c0044e3 from fd4o6 branch

(not cherry-picking it because the commit has other irrelevant changes)

[fd4o6] Added DHCPv4-over-DHCPv6 packet class

[4105] 4o6 configuration structure, unit-tests implemented

[trac4090] Add support for TokenSubstring

Add support and testing for TokenSubstring.  This token
takes three paramaters (string, start and length) and produces
a new string based on the original string.  It allows negative
values for start and length causing it to count from the end
of the string for start and to get characters before the start
point for length.

[3259] Ensure hooks example configuration files are copied on an install

[master] ChangeLog updated.

[master] Merge branch 'trac4081' (src/lib/eval added)

[4081] Added LOG4CPLUS_LIBS to src/lib/eval/tests/Makefile.am

[4081] Changes after review:

 - Copyright years corrected
 - Additional AM_CXXFLAGS for gcc added
 - LOG4CPLUS libs added to LDFLAGS
 - New unit-test for optionString for IPv6 packets added
 - Several comments in token.h clarified/corrected
 - Exception message extended slightly

[3259] Changes after review

[master] Changed string.h for cstring (cf #4089)

[3259] Updates to handle new syntax only

As some incompatible changes have already been made to the hooks
interface (removal of the setSkip method), it seems pointless to
support both old and new hoos-libraries syntax.  These modifications
remove support for the old syntax.

[3259] Corrections to the documentation

[3259] Fixed errors in .spec files.

[3259] Add test to check that mixed syntax element is rejected

[3259] Updating non-dhcpsrv test with new hooks library syntax