21189: Improve wording in config comments and logs.

Arvados-DCO-1.1-Signed-off-by: Tom Clegg <tom@curii.com>

diff --git a/lib/config/config.default.yml b/lib/config/config.default.yml
index 4fe75bd680614a09e0f644e6f14d36aa1715d2a4..844e67bfcc6bae721c8156c90fb8b29ca1ac3f05 100644 (file)
--- a/lib/config/config.default.yml
+++ b/lib/config/config.default.yml
@@ -642,8 +642,8 @@ Clusters:
       # block" requests to send to each keepstore server at a
       # time. Smaller values use less memory in keepstore and
       # keep-balance. Larger values allow more progress per
-      # keep-balance iteration. Setting both limits to zero amounts
-      # to a "dry run".
+      # keep-balance iteration. A zero value computes all of the
+      # needed changes but does not apply any.
       BalancePullLimit: 100000
       BalanceTrashLimit: 100000
 

diff --git a/services/keep-balance/server.go b/services/keep-balance/server.go
index bb7294c4ba1243763a3c774f61cd8953ab49b23b..b20144e3af9cae063f50f3ac5246e6f038ccbd2f 100644 (file)
--- a/services/keep-balance/server.go
+++ b/services/keep-balance/server.go
@@ -112,7 +112,7 @@ func (srv *Server) runForever(ctx context.Context) error {
        for {
                if srv.Cluster.Collections.BalancePullLimit < 1 && srv.Cluster.Collections.BalanceTrashLimit < 1 {
                        logger.Print("WARNING: Will scan periodically, but no changes will be committed.")
-                       logger.Print("=======  Consider using non-zero BalancePullLimit and BalanceTrashLimit configs.")
+                       logger.Print("=======  To commit changes, set BalancePullLimit and BalanceTrashLimit values greater than zero.")
                }
 
                if !dblock.KeepBalanceService.Check() {