]> git.ipfire.org Git - thirdparty/kea.git/commitdiff
[#1307] Addressed review comments
authorThomas Markwalder <tmark@isc.org>
Wed, 15 Sep 2021 16:02:32 +0000 (12:02 -0400)
committerThomas Markwalder <tmark@isc.org>
Fri, 17 Sep 2021 15:12:03 +0000 (11:12 -0400)
Minor clean ups in:

ChangeLog
doc/examples/kea4/all-keys.json
doc/examples/kea6/all-keys.json
doc/sphinx/arm/hooks-ha.rst
src/bin/dhcp4/tests/config_parser_unittest.cc

doc/examples/kea4/all-keys.json
doc/examples/kea6/all-keys.json
doc/sphinx/arm/hooks-ha.rst
src/bin/dhcp4/tests/config_parser_unittest.cc

index 1ad524ce39d87af60aed6ce35a88073a3561b49d..4863f0e7187fda615bdef3a41c79245c4f370cf5 100644 (file)
         // DHCPREQUESTs) that may be parked while waiting for hook library
         // work to complete, prior to a response (e.g. DHCPACK) being sent
         // back to the client.  A typical example is when kea-dhcp4 parks a
-        // DHCPREQUEST while it sends the requisite lease update(s) to its
-        // HA peer(s). The packet is unparked once the update(s) have been
-        // acknowledged.  This value limits the number of packets that can
-        // be held pending the updates. In times of heavy client traffic,
-        // this value can keep kea-dhcp4 from building an insurmountable
-        // backlog of updates.
+        // DHCPREQUEST while it sends the lease update(s) to its HA peer(s).
+        // The packet is unparked once the update(s) have been acknowledged.
+        // This value limits the number of packets that can be held pending
+        // the updates. In times of heavy client traffic, this value can keep
+        // kea-dhcp4 from building an insurmountable backlog of updates.
         "parked-packet-limit": 128,
 
         // List of global DHCP options that Kea DHCPv4 server assigns to the
index 3eb2486280e4667c878cf20fdbffd36049fe0755..9d2d028dd1a88db9f5c901c033152d73f1156ada 100644 (file)
         // REQUESTs,RENEWs...) that may be parked while waiting for
         // hook library work to complete, prior to a response (e.g. REPLY)
         // being sent back to the client.  A typical example is when kea-dhcp6
-        // parks a REQUEST while it sends the requisite lease update(s) to its
+        // parks a REQUEST while it sends the lease update(s) to its
         // HA peer(s). The packet is unparked once the update(s) have been
         // acknowledged.  This value limits the number of packets that can
         // be held pending the updates. In times of heavy client traffic,
index 337adc3e6e8e4cae7cadaef9146d224a3b2adb62..6544d04a6fdb59fa0f9056c6b5ddb6c2b619c897 100644 (file)
@@ -1602,10 +1602,10 @@ Parked Packet Limit
 ~~~~~~~~~~~~~~~~~~~
 
 Kea servers contain a mechanism by which the response to a client packet may
-be held, pending completion of Hook library work.  We refer to this as "parking"
+be held, pending completion of hook library work.  We refer to this as "parking"
 the packet.  The HA hook library makes use of this mechanism. When an HA server
 needs to send a lease update to its peer(s) to notify it of the change to the
-lease, it will "park" the client response until the peer acknowleges the lease
+lease, it will "park" the client response until the peer acknowledges the lease
 update.  At that point, the server will "unpark" the response and send it to the
 client.  This applies to client queries which cause lease changes such as
 DHCPREQUEST for DHCPv4 and REQUEST, RENEW, REBIND for DHCPv6. It does not apply
@@ -1619,8 +1619,8 @@ the limit is reached, the server will emit a log and drop any further responses
 until the parking spaces are available.
 
 In general, smaller values for the parking lot limit are likely to cause more
-drops but with response times. Larger values are likely to result in fewer drops
-but with longer response times.  Currently, the default value for
+drops but with shorter response times. Larger values are likely to result in
+fewer drops but with longer response times.  Currently, the default value for
 parked-packet-limit is 256.
 
 ::
index b87f48a5843858a39e0dac4f1fea7076bec09a4f..65b96676aaac480b2eaa7861513f0cedc066e71e 100644 (file)
@@ -7528,7 +7528,8 @@ TEST_F(Dhcp4ParserTest, multiThreadingSettings) {
                 << "  actual: " << *(cfg) << std::endl;
 }
 
-// Check whether it is possible to parked-packet-limit
+// Verify that parsing for the global parameter, parked-packet-limit,
+// is correct.
 TEST_F(Dhcp4ParserTest, parkedPacketLimit) {
     // Config without parked-packet-limit
     string config_no_limit = "{ " + genIfaceConfig() + "," +