[#2959] addressed comments

Author: Razvan Becheriu

doc/examples/kea4/reservations.json

@@ -48,10 +48,10 @@
 // to the dynamic pool. Kea is able to handle this case, but it is not
 // recommended from a performance perspective, as Kea would not only need to
 // check if a given address is free, but also whether it is reserved.
-// To avoid this check, one can set "reservations-out-of-pool", beside
-// the "reservations-in-subnet". If a subnet does not have reservations at all,
-// the reservation lookup can be skipped altogether by disabling
-// "reservations-in-subnet".
+// To avoid this check, one can set "reservations-out-of-pool" to true, beside
+// the "reservations-in-subnet" which should be also set to true. If a subnet
+// does not have reservations at all, the reservation lookup can be skipped
+// altogether by setting "reservations-in-subnet" to false.
 
 // Note that the second reservation is for an address which is within the
 // range of the pool of the dynamically allocated address. The server will
@@ -65,18 +65,20 @@
         "interface": "eth0",
         // This directive tells Kea that reservations may be made both in-pool
         // and out-of-pool. For improved performance, you may move all reservations
-        // out of the dynamic pool and set "reservations-out-of-pool".
+        // out of the dynamic pool and set "reservations-out-of-pool" to true.
         // Kea will then be able to skip querying for host reservations when
         // assigning leases from dynamic pool.
 
         // Specify whether the server should look up global reservations.
+        // Defaults to false.
         "reservations-global": false,
 
         // Specify whether the server should look up in-subnet reservations.
+        // Defaults to true.
         "reservations-in-subnet": true,
 
         // Specify whether the server can assume that all reserved addresses
-        // are out-of-pool.
+        // are out-of-pool. Defaults to false.
         // Ignored when reservations-in-subnet is false.
         // If specified, it is inherited by "shared-networks" and
         // "subnet4" levels.

doc/examples/kea6/reservations.json

@@ -47,18 +47,20 @@
 
         // This directive tells Kea that reservations may be made both in-pool
         // and out-of-pool. For improved performance, you may move all reservations
-        // out of the dynamic pool and set "reservations-out-of-pool".
+        // out of the dynamic pool and set "reservations-out-of-pool" to true.
         // Kea will then be able to skip querying for host reservations when
         // assigning leases from dynamic pool.
 
         // Specify whether the server should look up global reservations.
+        // Defaults to false.
         "reservations-global": false,
 
         // Specify whether the server should look up in-subnet reservations.
+        // Defaults to true.
         "reservations-in-subnet": true,
 
         // Specify whether the server can assume that all reserved addresses
-        // are out-of-pool.
+        // are out-of-pool. Defaults to false.
         // Ignored when reservations-in-subnet is false.
         // If specified, it is inherited by "shared-networks" and
         // "subnet6" levels.

doc/sphinx/arm/dhcp4-srv.rst

@@ -5324,27 +5324,27 @@ allocating or renewing a lease for the client.
 
 Configuration flags are:
 
--  ``reservations-in-subnet`` - enables in-pool host reservation types. This
-   setting is the default value, and is the safest and most flexible. However,
-   as all checks are conducted, it is also the slowest. It does not check
-   against global reservations.
-
--  ``reservations-out-of-pool`` - allows only out-of-pool host reservations.
-   With this setting in place, the server assumes that all host reservations are
-   for addresses that do not belong to the dynamic pool. Therefore, it can skip
-   the reservation checks when dealing with in-pool addresses, thus improving
-   performance. Do not use this mode if any reservations use in-pool addresses.
-   Caution is advised when using this setting; Kea does not sanity-check the
-   reservations against ``reservations-out-of-pool`` and misconfiguration may
-   cause problems.
+-  ``reservations-in-subnet`` - when set to true, it enables in-pool host
+   reservation types. This setting is the default value, and is the safest and
+   most flexible. However, as all checks are conducted, it is also the slowest.
+   It does not check against global reservations. This flag defaults to true.
+
+-  ``reservations-out-of-pool`` - when set to true, it allows only out-of-pool
+   host reservations. In this case the server assumes that all host reservations
+   are for addresses that do not belong to the dynamic pool. Therefore, it can
+   skip the reservation checks when dealing with in-pool addresses, thus
+   improving performance. Do not use this mode if any reservations use in-pool
+   addresses. Caution is advised when using this setting; Kea does not
+   sanity-check the reservations against ``reservations-out-of-pool`` and
+   misconfiguration may cause problems. This flag defaults to false.
 
 -  ``reservations-global`` - allows global host reservations. With this setting
    in place, the server searches for reservations for a client among the defined
    global reservations. If an address is specified, the server skips the
    reservation checks carried out in other modes, thus improving performance.
    Caution is advised when using this setting; Kea does not sanity-check the
    reservations when ``reservations-global`` is set, and misconfiguration may
-   cause problems.
+   cause problems. This flag defaults to  false.
 
 Note: disabling all flags disables host reservation support. As there are no
    reservations, the server skips all checks. Any reservations defined are

doc/sphinx/arm/dhcp6-srv.rst

@@ -4622,27 +4622,27 @@ allocating or renewing a lease for the client.
 
 Configuration flags are:
 
--  ``reservations-in-subnet`` - enables in-pool host reservation types. This
-   setting is the default value, and is the safest and most flexible. However,
-   as all checks are conducted, it is also the slowest. It does not check
-   against global reservations.
-
--  ``reservations-out-of-pool`` - allows only out-of-pool host reservations.
-   With this setting in place, the server assumes that all host reservations are
-   for addresses that do not belong to the dynamic pool. Therefore, it can skip
-   the reservation checks when dealing with in-pool addresses, thus improving
-   performance. Do not use this mode if any reservations use in-pool addresses.
-   Caution is advised when using this setting; Kea does not sanity-check the
-   reservations against ``reservations-out-of-pool`` and misconfiguration may
-   cause problems.
+-  ``reservations-in-subnet`` - when set to true, it enables in-pool host
+   reservation types. This setting is the default value, and is the safest and
+   most flexible. However, as all checks are conducted, it is also the slowest.
+   It does not check against global reservations. This flag defaults to true.
+
+-  ``reservations-out-of-pool`` - when set to true, it allows only out-of-pool
+   host reservations. In this case the server assumes that all host reservations
+   are for addresses that do not belong to the dynamic pool. Therefore, it can
+   skip the reservation checks when dealing with in-pool addresses, thus
+   improving performance. Do not use this mode if any reservations use in-pool
+   addresses. Caution is advised when using this setting; Kea does not
+   sanity-check the reservations against ``reservations-out-of-pool`` and
+   misconfiguration may cause problems. This flag defaults to false.
 
 -  ``reservations-global`` - allows global host reservations. With this setting
    in place, the server searches for reservations for a client among the defined
    global reservations. If an address is specified, the server skips the
    reservation checks carried out in other modes, thus improving performance.
    Caution is advised when using this setting; Kea does not sanity-check the
    reservations when ``reservations-global`` is set, and misconfiguration may
-   cause problems.
+   cause problems. This flag defaults to false.
 
 Note: disabling all flags disables host reservation support. As there are no
    reservations, the server skips all checks. Any reservations defined are