fix(backup): don't ignore uploadHost error

Fixes #3729
scylladb · Feb 26, 2024 · fa7dccf · fa7dccf
1 parent c215ad4
commit fa7dccf
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/pkg/service/backup/worker_snapshot.go b/pkg/service/backup/worker_snapshot.go
@@ -11,7 +11,7 @@ import (
 	"github.com/scylladb/scylla-manager/v3/pkg/util/timeutc"
 )
 
-func (w *worker) Snapshot(ctx context.Context, hosts []hostInfo, limits []DCLimit) (err error) {
+func (w *worker) Snapshot(ctx context.Context, hosts []hostInfo, limits []DCLimit) (err error) { // nolint: dupl
 	w.Logger.Info(ctx, "Taking snapshots...")
 	defer func(start time.Time) {
 		if err != nil {

diff --git a/pkg/service/backup/worker_upload.go b/pkg/service/backup/worker_upload.go
@@ -13,7 +13,7 @@ import (
 	"github.com/scylladb/scylla-manager/v3/pkg/util/timeutc"
 )
 
-func (w *worker) Upload(ctx context.Context, hosts []hostInfo, limits []DCLimit) (err error) {
+func (w *worker) Upload(ctx context.Context, hosts []hostInfo, limits []DCLimit) (err error) { // nolint: dupl
 	w.Logger.Info(ctx, "Uploading snapshot files...")
 	defer func(start time.Time) {
 		if err != nil {
@@ -25,7 +25,8 @@ func (w *worker) Upload(ctx context.Context, hosts []hostInfo, limits []DCLimit)
 
 	f := func(h hostInfo) error {
 		w.Logger.Info(ctx, "Uploading snapshot files on host", "host", h.IP)
-		if err := w.uploadHost(ctx, h); err == nil {
+		err := w.uploadHost(ctx, h)
+		if err == nil {
 			w.Logger.Info(ctx, "Done uploading snapshot files on host", "host", h.IP)
 		}
 		return err