Browse Source

[3981] Merge branch 'trac3981' of ssh://git.kea.isc.org/git/kea into trac3981

Tomek Mrugalski 9 years ago
parent
commit
079cf29ab9

+ 4 - 4
doc/guide/dhcp4-srv.xml

@@ -2727,13 +2727,13 @@ It is merely echoed by the server
       <para>
       <para>
         The server does not decrease assigned-addresses statistics
         The server does not decrease assigned-addresses statistics
         when DHCPDECLINE is received and processed successfully. While
         when DHCPDECLINE is received and processed successfully. While
-        technically declined address is not assigned anymore, the primary usage
-        of assigned-addresses statistic is to monitor pool utilization. Most
+        technically a declined address is no longer assigned, the primary usage
+        of the assigned-addresses statistic is to monitor pool utilization. Most
         people would forget to include declined-addresses in the calculation,
         people would forget to include declined-addresses in the calculation,
         and simply do assigned-addresses/total-addresses. This would have a bias
         and simply do assigned-addresses/total-addresses. This would have a bias
         towards under-representing pool utilization. As this has a
         towards under-representing pool utilization. As this has a
-        potential for major issues, we decided to decrease assigned
-        addresses not immediately after receiving DHCPDECLINE, but do
+        potential for major issues, we decided not to decrease assigned
+        addresses immediately after receiving DHCPDECLINE, but to do
         it later when we recover the address back to the available pool.
         it later when we recover the address back to the available pool.
       </para>
       </para>
 
 

+ 2 - 2
src/bin/dhcp4/dhcp4_srv.cc

@@ -1936,8 +1936,8 @@ Dhcpv4Srv::declineLease(const Lease4Ptr& lease, const std::string& descr) {
     StatsMgr::instance().addValue("declined-addresses", static_cast<int64_t>(1));
     StatsMgr::instance().addValue("declined-addresses", static_cast<int64_t>(1));
 
 
     // We do not want to decrease the assigned-addresses at this time. While
     // We do not want to decrease the assigned-addresses at this time. While
-    // technically declined address is not allocated anymore, the primary usage
-    // of assigned-addresses statistic is to monitor pool utilization. Most
+    // technically a declined address is no longer allocated, the primary usage
+    // of the assigned-addresses statistic is to monitor pool utilization. Most
     // people would forget to include declined-addresses in the calculation,
     // people would forget to include declined-addresses in the calculation,
     // and simply do assigned-addresses/total-addresses. This would have a bias
     // and simply do assigned-addresses/total-addresses. This would have a bias
     // towards under-representing pool utilization, if we decreased allocated
     // towards under-representing pool utilization, if we decreased allocated

+ 1 - 1
src/bin/dhcp4/dhcp4_srv.h

@@ -405,7 +405,7 @@ protected:
     /// @brief Process incoming DHCPDECLINE messages.
     /// @brief Process incoming DHCPDECLINE messages.
     ///
     ///
     /// This method processes incoming DHCPDECLINE. In particular, it extracts
     /// This method processes incoming DHCPDECLINE. In particular, it extracts
-    /// Requested IP Address option, check that the address really belongs to
+    /// Requested IP Address option, checks that the address really belongs to
     /// the client and if it does, calls @ref declineLease.
     /// the client and if it does, calls @ref declineLease.
     ///
     ///
     /// @param decline message received from client
     /// @param decline message received from client

+ 1 - 1
src/bin/dhcp4/tests/decline_unittest.cc

@@ -197,7 +197,7 @@ DeclineTest::acquireAndDecline(const std::string& hw_address_1,
     uint64_t after_global = declined_global->getInteger().first;
     uint64_t after_global = declined_global->getInteger().first;
 
 
     ASSERT_TRUE(lease);
     ASSERT_TRUE(lease);
-    // We check if the deline process was successful by checking if the
+    // We check if the decline process was successful by checking if the
     // lease is in the database and what is its state.
     // lease is in the database and what is its state.
     if (expected_result == SHOULD_PASS) {
     if (expected_result == SHOULD_PASS) {
         EXPECT_EQ(Lease::STATE_DECLINED, lease->state_);
         EXPECT_EQ(Lease::STATE_DECLINED, lease->state_);