From 7641d10184e6621be2bbf5f7c7bb748a73d810ee Mon Sep 17 00:00:00 2001 From: Freddy Date: Tue, 29 Nov 2022 15:56:03 -0700 Subject: [PATCH] Remove log line about server mgmt token init (#15610) * Remove log line about server mgmt token init Currently the server management token is only being bootstrapped in the primary datacenter. That means that servers on the secondary datacenter will never have this token available, and would log this line any time a token is resolved. Bootstrapping the token in secondary datacenters will be done in a follow-up. * Add changelog entry --- .changelog/15610.txt | 3 +++ agent/consul/acl_server.go | 1 - 2 files changed, 3 insertions(+), 1 deletion(-) create mode 100644 .changelog/15610.txt diff --git a/.changelog/15610.txt b/.changelog/15610.txt new file mode 100644 index 000000000..6e8cfad16 --- /dev/null +++ b/.changelog/15610.txt @@ -0,0 +1,3 @@ +```release-note:bug +acl: avoid debug log spam in secondary datacenter servers due to management token not being initialized. +``` \ No newline at end of file diff --git a/agent/consul/acl_server.go b/agent/consul/acl_server.go index 8e14d502a..6fdfa1dc8 100644 --- a/agent/consul/acl_server.go +++ b/agent/consul/acl_server.go @@ -116,7 +116,6 @@ func (s *serverACLResolverBackend) IsServerManagementToken(token string) bool { return false } if mgmt == "" { - s.logger.Debug("server management token has not been initialized") return false } return subtle.ConstantTimeCompare([]byte(mgmt), []byte(token)) == 1