Skip to content

Commit

Permalink
Fix clearscroll request fmt string (#15416) (#15435)
Browse files Browse the repository at this point in the history
clearscroll log should use Warnf instead of Warn.

(cherry picked from commit e7deaa4)

Co-authored-by: Carson Ip <carsonip@users.noreply.github.com>
  • Loading branch information
mergify[bot] and carsonip authored Jan 28, 2025
1 parent 8df42cd commit c19d54b
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion internal/agentcfg/elasticsearch.go
Original file line number Diff line number Diff line change
Expand Up @@ -262,7 +262,7 @@ func (f *ElasticsearchFetcher) clearScroll(ctx context.Context, scrollID string)
}

if resp.IsError() {
f.logger.Warn("clearscroll request returned error: %s", resp.Status())
f.logger.Warnf("clearscroll request returned error: %s", resp.Status())
}

resp.Body.Close()
Expand Down
2 changes: 1 addition & 1 deletion internal/sourcemap/metadata_fetcher.go
Original file line number Diff line number Diff line change
Expand Up @@ -196,7 +196,7 @@ func (s *MetadataESFetcher) clearScroll(ctx context.Context, scrollID string) {
}

if resp.IsError() {
s.logger.Warn("clearscroll request returned error: %s", resp.Status())
s.logger.Warnf("clearscroll request returned error: %s", resp.Status())
}

resp.Body.Close()
Expand Down

0 comments on commit c19d54b

Please sign in to comment.