respecting ignoredPath

Signed-off-by: Alan Protasio <approtas@amazon.com>

pkg/objstore/objstore.go

@@ -12,6 +12,7 @@ import (
 	"path"
 	"path/filepath"
 	"strings"
+	"sync"
 	"time"
 
 	"github.com/go-kit/log"
@@ -263,13 +264,17 @@ func DownloadDir(ctx context.Context, logger log.Logger, bkt BucketReader, origi
 	g, ctx := errgroup.WithContext(ctx)
 
 	var downloadedFiles []string
+	var m sync.Mutex
+
 	err := bkt.Iter(ctx, src, func(name string) error {
 		g.Go(func() error {
 			dst := filepath.Join(dst, filepath.Base(name))
 			if strings.HasSuffix(name, DirDelim) {
 				if err := DownloadDir(ctx, logger, bkt, originalSrc, name, dst, ignoredPaths...); err != nil {
 					return err
 				}
+				m.Lock()
+				defer m.Unlock()
 				downloadedFiles = append(downloadedFiles, dst)
 				return nil
 			}
@@ -279,7 +284,15 @@ func DownloadDir(ctx context.Context, logger log.Logger, bkt BucketReader, origi
 					return nil
 				}
 			}
-			return DownloadFile(ctx, logger, bkt, name, dst)
+			if err := DownloadFile(ctx, logger, bkt, name, dst); err != nil {
+				return err
+			}
+
+			m.Lock()
+			defer m.Unlock()
+			downloadedFiles = append(downloadedFiles, dst)
+
+			return nil
 		})
 		downloadedFiles = append(downloadedFiles, dst)
 		return nil