Skip to content

Commit

Permalink
[Fix] Replace unsupported %v macro in logs (thanos-io#3847)
Browse files Browse the repository at this point in the history
Signed-off-by: Stéphane Brillant <[email protected]>
  • Loading branch information
StephaneBrillant authored and Andre Branchizio committed Mar 11, 2021
1 parent f8ff622 commit 9736bdd
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion cmd/thanos/query.go
Original file line number Diff line number Diff line change
Expand Up @@ -556,7 +556,7 @@ func removeDuplicateStoreSpecs(logger log.Logger, duplicatedStores prometheus.Co
for _, spec := range specs {
addr := spec.Addr()
if _, ok := set[addr]; ok {
level.Warn(logger).Log("msg", "Duplicate store address is provided - %v", addr)
level.Warn(logger).Log("msg", "Duplicate store address is provided", "addr", addr)
duplicatedStores.Inc()
}
set[addr] = spec
Expand Down
2 changes: 1 addition & 1 deletion cmd/thanos/rule.go
Original file line number Diff line number Diff line change
Expand Up @@ -721,7 +721,7 @@ func removeDuplicateQueryEndpoints(logger log.Logger, duplicatedQueriers prometh
deduplicated := make([]*url.URL, 0, len(urls))
for _, u := range urls {
if _, ok := set[u.String()]; ok {
level.Warn(logger).Log("msg", "duplicate query address is provided - %v", u.String())
level.Warn(logger).Log("msg", "duplicate query address is provided", "addr", u.String())
duplicatedQueriers.Inc()
continue
}
Expand Down

0 comments on commit 9736bdd

Please sign in to comment.