Receive: upgrading logs for failed uploads to error (thanos-io#6427)

* FIX: upgrading log for failed upload to error Signed-off-by: Victor Fernandes <victorhbfernandes@gmail.com> * docs: added changelog entry Signed-off-by: Victor Fernandes <victorhbfernandes@gmail.com> --------- Signed-off-by: Victor Fernandes <victorhbfernandes@gmail.com>
fpetkovski · Jun 27, 2023 · ab1fbbb · ab1fbbb
1 parent b509621
commit ab1fbbb
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -22,7 +22,7 @@ We use *breaking :warning:* to mark changes that are not backward compatible (re
 - [#6352](https://github.com/thanos-io/thanos/pull/6352) Store: Expose store gateway query stats in series response hints.
 
 ### Fixed
-
+- [#6427](https://github.com/thanos-io/thanos/pull/6427) Receive: increasing log level for failed uploads to error
 - [#6172](https://github.com/thanos-io/thanos/pull/6172) query-frontend: return JSON formatted errors for invalid PromQL expression in the split by interval middleware.
 - [#6171](https://github.com/thanos-io/thanos/pull/6171) Store: fix error handling on limits.
 - [#6183](https://github.com/thanos-io/thanos/pull/6183) Receiver: fix off by one in multitsdb flush that will result in empty blocks if the head only contains one sample

diff --git a/cmd/thanos/receive.go b/cmd/thanos/receive.go
@@ -710,12 +710,12 @@ func startTSDBAndUpload(g *run.Group,
  case <-uploadC:
  // Upload on demand.
  if err := upload(ctx); err != nil {
- level.Warn(logger).Log("msg", "on demand upload failed", "err", err)
+ level.Error(logger).Log("msg", "on demand upload failed", "err", err)
  }
  uploadDone <- struct{}{}
  case <-tick.C:
  if err := upload(ctx); err != nil {
- level.Warn(logger).Log("msg", "recurring upload failed", "err", err)
+ level.Error(logger).Log("msg", "recurring upload failed", "err", err)
  }
  }
  }