Update based on PR feedback

Signed-off-by: dkwon17 <dakwon@redhat.com>

Author: dkwon17

project-clone/internal/git/setup.go

@@ -64,7 +64,7 @@ func doInitialGitClone(project *dw.Project) error {
 		}
 	}
 	if cloneErr != nil {
-		return fmt.Errorf("failed to clone project: %s", cloneErr)
+		return fmt.Errorf("failed to clone project: %w", cloneErr)
 	}
 
 	if project.Attributes.Exists(internal.ProjectSparseCheckout) {

project-clone/internal/global.go

@@ -37,6 +37,7 @@ const (
 	publicCertsDir            = "/public-certs"
 	cloneRetriesEnvVar        = "PROJECT_CLONE_RETRIES"
 	defaultCloneRetries       = 3
+	maxCloneRetries           = 10
 	BaseRetryDelay            = 1 * time.Second
 )
 
@@ -70,6 +71,9 @@ func init() {
 		parsed, err := strconv.Atoi(val)
 		if err != nil || parsed < 0 {
 			log.Printf("Invalid value for %s: %q, using default (%d)", cloneRetriesEnvVar, val, defaultCloneRetries)
+		} else if parsed > maxCloneRetries {
+			log.Printf("Value for %s (%d) exceeds maximum (%d), using maximum", cloneRetriesEnvVar, parsed, maxCloneRetries)
+			CloneRetries = maxCloneRetries
 		} else {
 			CloneRetries = parsed
 		}