[arvados] updated: 2.1.0-2959-g14f662537

[git repository hosting]

Summary of changes:
 lib/diagnostics/cmd.go | 4 ++++
 1 file changed, 4 insertions(+)

       via  14f6625379992bc3ee054ad419095b476a1c4284 (commit)
      from  58afa4202a5ff084fe7eee1a5274e02071858ba5 (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 14f6625379992bc3ee054ad419095b476a1c4284
Author: Tom Clegg <tom at curii.com>
Date:   Tue Oct 18 13:23:26 2022 -0400

    19377: Fix diag sanity check. Suggest "check -yaml" on health fail.
    
    Arvados-DCO-1.1-Signed-off-by: Tom Clegg <tom at curii.com>

diff --git a/lib/diagnostics/cmd.go b/lib/diagnostics/cmd.go
index 5799ceb9a..888503b55 100644
--- a/lib/diagnostics/cmd.go
+++ b/lib/diagnostics/cmd.go
@@ -157,12 +157,16 @@ func (diag *diagnoser) runtests() {
 		}
 		if cluster.SystemRootToken != os.Getenv("ARVADOS_API_TOKEN") {
 			diag.infof("skipping because provided token is not SystemRootToken")
+			return nil
 		}
 		agg := &health.Aggregator{Cluster: cluster}
 		resp := agg.ClusterHealth()
 		for _, e := range resp.Errors {
 			diag.errorf("health check: %s", e)
 		}
+		if len(resp.Errors) > 0 {
+			diag.infof("consider running `arvados-server check -yaml` for a comprehensive report")
+		}
 		diag.verbosef("reported clock skew = %v", resp.ClockSkew)
 		reported := map[string]bool{}
 		for _, result := range resp.Checks {

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


hooks/post-receive
--