fix(s3): check for errors when finding artifacts (#689)

Signed-off-by: Alan Clucas <[email protected]>
argoproj · Jul 20, 2024 · 42ec18a · 42ec18a
1 parent 88bae10
commit 42ec18a
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/s3/s3.go b/s3/s3.go
@@ -252,7 +252,11 @@ func generatePutTasks(keyPrefix, rootPath string) chan uploadTask {
 	rootPath = filepath.Clean(rootPath) + string(os.PathSeparator)
 	uploadTasks := make(chan uploadTask)
 	go func() {
-		_ = filepath.Walk(rootPath, func(localPath string, fi os.FileInfo, _ error) error {
+		_ = filepath.Walk(rootPath, func(localPath string, fi os.FileInfo, err error) error {
+			if err != nil {
+				log.WithFields(log.Fields{"localPath": localPath}).Error("Failed to walk artifacts path", err)
+				return err
+			}
 			relPath := strings.TrimPrefix(localPath, rootPath)
 			if fi.IsDir() {
 				return nil