GitLab

This is trivial fix so as I am committing it on my own discretion. Also,
this fix had already been reviewed and applied on master branch but
apparently disappeared after the recent git migration.

Also, removed multiple references to DHCP4_CONFIG_LOAD_FAIL from the
DHCPv4 server and DHCP6_CONFIG_LOAD_FAIL from the DHCPv6 server.

The updated code uses interface index to match the interface detected by
the test with the interface found by the IfaceMgr. This is to avoid issues
on Linux whereby Netlink-based implementation doesn't return the virtual
interface names, but rather physical interface names.

 Thanks to Jiri Popelka for the fix

This is required on NetNSD and OpenBSD so as multiple sockets can be bound
to in6addr_any and dhcp server port.

Also, fixed selection of the unicast addresses for receiving DHCP packets.

Also, added better documentation for the IfaceCfg class.

This change affects DHCP servers and D2.

Changes include documenting the constants used for selection of parameters
to be reported by getConfigSummary and a todo.

Removed the IfaceMgr.detectIfaces_linux test as it is covered in the
IfaceMgr.detectIfaces test already.

The original version of this test incorrectly matched the names of the
interfaces detected by the IfaceMgr and those detected by the unit test.
As a result, the interfaces with similar names (e.g. virbr0 and virbr0-nic)
where compared for the equality of indexes, addresses and flags.

It was failing on a few different systems.
Okay to disable via jabber.
The ticket is #3510.

 - Daemon::getVersion() added
 - exists => exits in docs
 - fixed comment in dhcp_test_lib.sh.in