[#1654] Addressed comments, rebasing

diff --git a/doc/sphinx/arm/hooks-lease-cmds.rst b/doc/sphinx/arm/hooks-lease-cmds.rst
index 98815d5a2dca2b58f2ac04fe8118ec21e7151293..a818230965ac49d00d4f11887664a858ebb5023c 100644 (file)
--- a/doc/sphinx/arm/hooks-lease-cmds.rst
+++ b/doc/sphinx/arm/hooks-lease-cmds.rst
@@ -1023,9 +1023,11 @@ backend and write the lease database into a CSV file. They take the path
 of the file as the ``filename`` argument. If the specified output file
 is the same as the configured memfile one the backend close and reopen
 the file in an attempt to synchronize both file and in memory images
-of the lease database.
+of the lease database. The previous file is renamed by appending ``.bak``
+to its name.
 
 .. note::
 
    These commands do not replace the LFC mechanism: they should be used
-   only in exceptional circumstances.
+   only in exceptional circumstances, such as when recovering after
+   running out of disk space.

diff --git a/src/hooks/dhcp/lease_cmds/tests/lease_cmds4_unittest.cc b/src/hooks/dhcp/lease_cmds/tests/lease_cmds4_unittest.cc
index a5e276f10a1479c3f64eb19bb8ab090155d6d753..85b3c8a0456a482d7abf99f007f3c9257edffeb9 100644 (file)
--- a/src/hooks/dhcp/lease_cmds/tests/lease_cmds4_unittest.cc
+++ b/src/hooks/dhcp/lease_cmds/tests/lease_cmds4_unittest.cc
@@ -3139,6 +3139,9 @@ void Lease4CmdsTest::testLease4ConflictingUpdate() {
 }
 
 void Lease4CmdsTest::testLease4Write() {
+    // lease4-write negative tests. Positive tests are in the
+    // memfile_lease_mgr_unittest.cc file.
+
     // Initialize lease manager (false = v4, false = don't add leases)
     initLeaseMgr(false, false);