From 332c7134c72c73478ac57b7f5cae087706ed6c5f Mon Sep 17 00:00:00 2001 From: Peter Mattis Date: Thu, 30 Aug 2018 21:31:14 -0400 Subject: [PATCH] server: fix a confusing start message Include both bootstrapped (existing) and empty (new) engines in the start message. Previously the message was only include bootstrapped engines which was confusing when a node first joined a cluster. See #27599 Release note: None --- pkg/server/node.go | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/pkg/server/node.go b/pkg/server/node.go index 261c2cf58e96..50787a79c1a4 100644 --- a/pkg/server/node.go +++ b/pkg/server/node.go @@ -543,7 +543,9 @@ func (n *Node) start( // bumped immediately, which would be possible if gossip got started earlier). n.startGossip(ctx, n.stopper) - log.Infof(ctx, "%s: started with %v engine(s) and attributes %v", n, bootstrappedEngines, attrs.Attrs) + allEngines := append([]engine.Engine(nil), bootstrappedEngines...) + allEngines = append(allEngines, emptyEngines...) + log.Infof(ctx, "%s: started with %v engine(s) and attributes %v", n, allEngines, attrs.Attrs) return nil }