[ARVADOS] updated: 1.3.0-1671-g5ffded80f

Git user git at public.curoverse.com
Thu Oct 3 18:46:15 UTC 2019


Summary of changes:
 doc/admin/upgrading.html.textile.liquid | 2 +-
 lib/config/deprecated.go                | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

       via  5ffded80f04bc1b38574b0a4eee54c6ceb9b9112 (commit)
      from  e86ee860d99036ff4ac61f6635b738f3a408e446 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.


commit 5ffded80f04bc1b38574b0a4eee54c6ceb9b9112
Author: Eric Biagiotti <ebiagiotti at veritasgenetics.com>
Date:   Thu Oct 3 14:45:47 2019 -0400

    14714: Improves error message when using the legacy keep-balance config
    
    Arvados-DCO-1.1-Signed-off-by: Eric Biagiotti <ebiagiotti at veritasgenetics.com>

diff --git a/doc/admin/upgrading.html.textile.liquid b/doc/admin/upgrading.html.textile.liquid
index 2ec636a63..cfb36cef8 100644
--- a/doc/admin/upgrading.html.textile.liquid
+++ b/doc/admin/upgrading.html.textile.liquid
@@ -58,7 +58,7 @@ h4. Keep-balance configuration migration
 
 You can no longer specify types of keep services to balance via the @KeepServiceTypes@ config option in the legacy config at @/etc/arvados/keep-balance/keep-balance.yml at . If you are still using the legacy config and @KeepServiceTypes@ has a value other than "disk", keep-balance will produce an error.
 
-You can no longer specify individual keep services to balance via the @config.KeepServiceList@ command line option or @KeepServiceList@ legacy config option. Instead, keep-balance operates on all configured volumes. If you are still using the legacy config, @KeepServiceList@ should be removed or keep-balance will produce an error.
+You can no longer specify individual keep services to balance via the @config.KeepServiceList@ command line option or @KeepServiceList@ legacy config option. Instead, keep-balance will operate on all volumes of service type @disk@ as reported by the @arv keep_service list@ command. If you are still using the legacy config, @KeepServiceList@ should be removed or keep-balance will produce an error.
 
 Please see the "config migration guide":{{site.baseurl}}/admin/config-migration.html and "keep-balance install guide":{{site.baseurl}}/install/install-keep-balance.html for more details.
 
diff --git a/lib/config/deprecated.go b/lib/config/deprecated.go
index e6b06c131..22eed080a 100644
--- a/lib/config/deprecated.go
+++ b/lib/config/deprecated.go
@@ -532,7 +532,7 @@ func (ldr *Loader) loadOldKeepBalanceConfig(cfg *arvados.Config) error {
 		cluster.API.KeepServiceRequestTimeout = *oc.RequestTimeout
 	}
 
-	msg := "The %s configuration option is no longer supported. Please remove it from your configuration file. Keep-balance will operate on all configured volumes."
+	msg := "The %s configuration option is no longer supported. Please remove it from your configuration file. See the keep-balance upgrade notes at https://doc.arvados.org/admin/upgrading.html for more details."
 
 	// If the keep service type provided is "disk" silently ignore it, since
 	// this is what ends up being done anyway.

-----------------------------------------------------------------------


hooks/post-receive
-- 




More information about the arvados-commits mailing list