diff --git a/.changelog/8685.txt b/.changelog/8685.txt new file mode 100644 index 0000000000..17ae094bea --- /dev/null +++ b/.changelog/8685.txt @@ -0,0 +1,3 @@ +```release-note:bug +fixed a bug that caused logs to be flooded with `[WARN] agent.router: Non-server in server-only area` +``` diff --git a/agent/router/router.go b/agent/router/router.go index 243082a0ef..027303bea2 100644 --- a/agent/router/router.go +++ b/agent/router/router.go @@ -536,10 +536,13 @@ func (r *Router) GetDatacentersByDistance() ([]string, error) { for _, m := range info.cluster.Members() { ok, parts := metadata.IsConsulServer(m) if !ok { - r.logger.Warn("Non-server in server-only area", - "non_server", m.Name, - "area", areaID, - ) + if areaID != types.AreaLAN { + r.logger.Warn("Non-server in server-only area", + "non_server", m.Name, + "area", areaID, + "func", "GetDatacentersByDistance", + ) + } continue } @@ -547,6 +550,7 @@ func (r *Router) GetDatacentersByDistance() ([]string, error) { r.logger.Debug("server in area left, skipping", "server", m.Name, "area", areaID, + "func", "GetDatacentersByDistance", ) continue } @@ -607,10 +611,13 @@ func (r *Router) GetDatacenterMaps() ([]structs.DatacenterMap, error) { for _, m := range info.cluster.Members() { ok, parts := metadata.IsConsulServer(m) if !ok { - r.logger.Warn("Non-server in server-only area", - "non_server", m.Name, - "area", areaID, - ) + if areaID != types.AreaLAN { + r.logger.Warn("Non-server in server-only area", + "non_server", m.Name, + "area", areaID, + "func", "GetDatacenterMaps", + ) + } continue } @@ -618,6 +625,7 @@ func (r *Router) GetDatacenterMaps() ([]structs.DatacenterMap, error) { r.logger.Debug("server in area left, skipping", "server", m.Name, "area", areaID, + "func", "GetDatacenterMaps", ) continue }