[4259] Address review comments, part II

    Fixed minor typos.

[4259] Addressed review comments

    Moved string/enum conversion functions inside D2ClientConfig

    Changed underscores to hypens in replace-client-name values and changed
    all literal values to lower case.

    Fixed minor wording issues in admin guide and new log messages.

    Added commentary and extra error logging to unit tests

    Added test case for invalid value of replace-client-name

[4259] Updated admin guide and spec files

    Revamped discussion of FQDN name generation and use of the
    replace-client-name parameter in the v4 and v6 guides.

[4259] kea-dhcp6 now supports replace-client-name modes

    src/bin/dhcp6/dhcp6_messages.mes
        - Added new log message, DHCP6_DDNS_SUPPLY_FQDN

    src/bin/dhcp6/dhcp6_srv.cc
        - Dhcpv6Srv::processClientFqdn() - modified to support the name
        replacement modes

    src/bin/dhcp6/tests/fqdn_unittest.cc
        - FqdnDhcpv6SrvTest::testReplaceClientNameMode() new method which tests
        a server's handling of a single client packet for a given
        replace-client-name mode.

        - TEST_F(FqdnDhcpv6SrvTest, replaceClientNameModeTest) - new test which
        exercises the permutations of client packets and replace-client-name
        modes.

[4259] kea-dhcp4 now supports replace-client-name modes

    src/bin/dhcp4/dhcp4_messages.mes
        - Added new log message DHCP4_SUPPLY_HOSTNAME

    src/bin/dhcp4/dhcp4_srv.cc
        - Dhcpv4Srv::processClientName() - pushed lack of host option in the
        client request down into processHostoption()

        - Dhcpv4Srv::processHostnameOption() - modified to support the new name
        replacement modes

    src/bin/dhcp4/tests/fqdn_unittest.cc
        - NameDhcpv4SrvTest:: testReplaceClientNameMode() - new method which
        tests a server's handling of a single client packet for a given
        replace-client-name mode.

        - TEST_F(NameDhcpv4SrvTest, replaceClientNameModeTest) - new test which
        exercises the permutations of client packets and replace-client-name
        modes.

[4259] Converted replace-client-name from boolean to enumeration

    The D2 client configuration parameter, replace-client-name, was
    changed from a boolean, to an enumerated list of modes:

        "NEVER" - do not alter or supply the client name
        "ALWAYS" - always replace the client name, or supply it if
         not sent by the client
        "WHEN_PRESENT" - replace the client name sent by the client,
         do not supply one otherwise
        "WHEN_NOT_PRESENT" - supply the client name only if one was
        not sent by the client

    src/lib/dhcpsrv/d2_client_cfg.cc
    src/lib/dhcpsrv/d2_client_cfg.h
        Added the D2ClientConfig::ReplaceClientNameMode enumeration.

    src/lib/dhcpsrv/parsers/dhcp_parsers.cc
        - Alter replace-client-name parsing to parse the mode labels.
        - Maps boolean literals true and false to RCM_WHEN_PRESENT and
        RCM_NEVER respectively

    src/bin/dhcp4/dhcp6_srv.cc
    src/bin/dhcp4/dhcp4_srv.cc
    src/lib/dhcpsrv/d2_client_mgr.h
        - Adapted to use the enumeration

    src/bin/dhcp4/tests/config_parser_unittest.cc
    src/bin/dhcp4/tests/fqdn_unittest.cc
    src/bin/dhcp6/tests/config_parser_unittest.cc
    src/bin/dhcp6/tests/fqdn_unittest.cc
    src/lib/dhcpsrv/tests/cfgmgr_unittest.cc
    src/lib/dhcpsrv/tests/d2_client_unittest.cc
    src/lib/dhcpsrv/tests/d2_udp_unittest.cc
    src/lib/dhcpsrv/tests/dhcp_parsers_unittest.cc
        - Alter existing tests to use the enumeration. Note false was replaced
        with RCM_NEVER and true with RCM_WHEN_PRESENT

[4259] Add unit tests to ensure StringParser can parse booleans

    Added checks to ensure that StringParser can handle being given
    a boolean literal (i.e. unquoted true or false)

[master] Added ChangeLog entry for #4302.

[master] Merge branch 'trac4302_rebase'

[4302] Address further review comments.

- cppcheck issues
- Unique index for prefix and prefix length.

[4302] Additional indexes added to search for hosts in MySQL.

[4302] MySQL database upgraded to version 4.2.

- Added new table host_identifier_type
- Added foreign key on hosts table referencing
  host_identifier_type
- Indexes on hosts table are now unique

[4302] Addressed review comments.

- Rename GET_HOST_HWADDR_DUID to GET_HOST_DHCPID
- Fixes and clarifications in doxygen documentation

[4302] Use new functions to retrieve reservations by any identifier.

[4302] Refactored host data source classes.

The updated API exposes functions which retrieve hosts
by any identifier and identifier type, rather than DUID
or HW address.

[master] Eliminate doxygen warnings after #4212 and #3696.

Committing on my discretion and with no objections on jabber.

[master] Added ChangeLog entry for #4212.

[master] Updated git hash

[master] Finished merge of trac4267 (final run_one())

[master] Merged trac4267 (final run_one())

[4267] Addressed comments

[master] Added ChangeLog entry 1096 for Trac 4316

[master] Primary lease events now log at INFO level

    Merges in branch 'trac4316'

[master] Merge branch 'trac4212'

[4316] Changed primary lease event logging from DEBUG to INFO

src/bin/dhcp4/dhcp4_messages.mes
    - Replaced the word "debug" with "informational" where appropriate

src/bin/dhcp4/dhcp4_srv.cc
    - Changed several lease4_logger messages from DEBUG to INFO
    DHCP4_INIT_REBOOT
    DHCP4_LEASE_ADVERT
    DHCP4_LEASE_ALLOC
    DHCP4_RELEASE
    DHCP4_DECLINE_LEASE

src/bin/dhcp6/dhcp6_messages.mes
    - Replaced the word "debug" with "informational" where appropriate

    - Added two new messages:
    DHCP6_LEASE_RENEW
    DHCP6_PD_LEASE_RENEW

src/bin/dhcp6/dhcp6_srv.cc
    - Changed several lease6_logger messages from DEBUG to INFO
    DHCP6_LEASE_ADVERT
    DHCP6_LEASE_ALLOC
    DHCP6_PD_LEASE_ADVERT
    DHCP6_PD_LEASE_ALLOC
    DHCP6_RELEASE_NA
    DHCP6_RELEASE_PD
    DHCP6_DECLINE_LEASE

    - Dhcpv6Srv::extendIA_NA() - added new log for DHCP6_LEASE_RENEW
    - Dhcpv6Srv::extendIA_PD() - added new log for DHCP6_PD_LEASE_RENEW

[4212] Addressed review comments.

[master] Updated git hash

[master] Finished merge of trac4326 (small Coverity fixes)

[master] Merged trac4326 (small Coverity fixes)

[4326] Changed ULL by an inline cast

[4326] untabified

[master] Added ChangeLog entry for #4339.

[master] Merge branch 'trac4339a'

[4339a] grammar

[4339] Addressed review comments.

- Replace rand() with random()
- Use i++ rather than ++i

[master] Updated git hash

[master] Finished merge of trac4306 (overflow in pkt4::set{File,Name}())

[master] Merged trac4306 (overflow in pkt4::set{File,Name}())

[4306] Added comments about uninitilaized too big arrays

[4339] Fixed issues with hostname generation in new unit test.

[4339] Corrected issues with updating lease indexes in Memfile.

[4339] Unit-test for updating lease indexes implemented.

Also updated tests for Lease6::toText function.

[4326] Fixed CID 1341464

[4326] Fixed CID 132738[1-4]

[4306] Added/improved unit tests

[master] Removed linking with gtest in libdhcpsrvtest.

This change was okayed on jabber by Thomas. The issue
occurred when static linking was enabled on OSX.

Linking a static lib from statically linked lib causes
various issues and isn't generally considered a good
idea.

[master] Added ChangeLog entry for #4319.

[master] Merge branch 'trac4319'

[4319] Updated comments for empty options in host tests.

[master] Updated git hash

[master] Finished merge of trac4307 (uncaught exception in d2 and other server main())

[master] Merged trac4307 (uncaught exception in d2 server main())

[master] Updated git hash

[master] regen flex/bison

[master] Fixing some .gitignore

[master] Finished merge of trac4232a (IP address literal)

[4319] Addressed some review comments.

- use unsigned int in for loops
- NULL pointer rather than NULL in function descriptions

[master] Fixed Coverity issue in alloc_engine.cc

    Merged in Trac #4309.

[trac4309] Fix some typos

[trac4232a] Typo in the classify text

[master] Updated git hash (and regen flex/bison)

regen flex/bison

[master] Finishing merge of trac4313 (relay4[xxx].exists)

[master] Merged trac4313 (relay4[xxx].exists)

[4313] spelling

[4313] Added parameter descriptions in two context unit-tests functions.

[4232a] Addressed comments

[master] Merge branch 'trac4226' (Pkt4 exception clean up)

[4319] Added additional configuration parser tests.

New test check that it is possible to specify a
non-standard option within a host declaration.

[4319] Added new config parser unit test for DHCPv4 and DHCPv6.

These tests check that it is possible to specify options
for hosts.

[master] Updated git hash

[master] Finished merge of trac4310 (LOG_FATAL() double errors)

[master] Merged trac4310 (LOG_FATAL() double errors)

[master] Updated git hash

[master] Finished merge of rac4291 (empty option vs csv-format)

[master] Merged trac4291 (empty option vs csv-format)

[4232a] Grammar (by Stephen)

[4319] Extend host reservation parser to parse options.

[4309] Fixed Coverity issue 1341466e in alloc_engine.cc

src/lib/dhcpsrv/alloc_engine.cc
 - AllocEngine::reclaimExpiredLease(const Lease6Ptr& lease,
                                 const DbReclaimMode& reclaim_mode,
                                 const CalloutHandlePtr& callout_handle)

   Simplified logic for flagging a lease for removal.  Renamed "remove_tmp"
   to "remove_lease" for clarity.

 - AllocEngine::reclaimExpiredLease(const Lease4Ptr& lease,
                                 const DbReclaimMode& reclaim_mode,
                                 const CalloutHandlePtr& callout_handle)

   Simplified logic for flagging a lease for removal.  Renamed "remove_tmp"
   to "remove_lease" for clarity.

[4326] Fixed Coverity CID 1327378 (needs a second view)

[4326] Fixed Coverity CID 1232320

[4326] Fixed Coverity CID 1232272

[4326] Fixed Coverity CID 1232271

[4326] Addressed coverity CID 1202723

[4326] copyright

[4326] Addressed coverity CID 1202661

[master] Added ChangeLog entry for #3571.

[master] Merge branch 'trac3571'

[3571] Address review comments.

Corrected mixed up use of DHCPv4 vs DHCPv6 and the
dhcp4 vs dhcp6 option space names.

[master] Add commit number to change log entry for 4292

[master] Add comment and change log entry for 4292

Merge branch 'trac4292' Copy BOOTP flags from requets to responses

[4313] Added TokenOption::getRepresention()

[4313] Improved relay4[code].exists comment

[4313] untabify

[4313] Explained relay4 shares option representation types

[4313] Added a comment about Token subtype order

[4313] Added a comment about token order

[4212] Compilation fix

[master] Updated git hash