Yarn - Rewording log messages

Author: agrasth

utils/retryexecutor.go

@@ -4,9 +4,11 @@ import (
 	"context"
 	"errors"
 	"fmt"
-	"github.com/jfrog/jfrog-client-go/utils/errorutils"
+	"strings"
 	"time"
 
+	"github.com/jfrog/jfrog-client-go/utils/errorutils"
+
 	"github.com/jfrog/jfrog-client-go/utils/log"
 )
 
@@ -81,12 +83,36 @@ func (runner *RetryExecutor) getTimeoutErrorMsg() string {
 }
 
 func (runner *RetryExecutor) LogRetry(attemptNumber int, err error) {
-	message := fmt.Sprintf("%s(Attempt %v)", runner.LogMsgPrefix, attemptNumber+1)
+	message := fmt.Sprintf("%s(Attempt %v of %v)", runner.LogMsgPrefix, attemptNumber+1, runner.MaxRetries+1)
 	if runner.ErrorMessage != "" {
 		message = fmt.Sprintf("%s - %s", message, runner.ErrorMessage)
 	}
 	if err != nil {
-		message = fmt.Sprintf("%s: %s", message, err.Error())
+		errStr := err.Error()
+		switch {
+		case strings.Contains(errStr, "i/o timeout"):
+			message = fmt.Sprintf("%s: Temporary network timeout occurred, retrying...", message)
+		case strings.Contains(errStr, "connection refused"):
+			message = fmt.Sprintf("%s: Connection refused - server may be temporarily unavailable, retrying...", message)
+		case strings.Contains(errStr, "no such host"):
+			message = fmt.Sprintf("%s: DNS resolution failed - host may be temporarily unavailable, retrying...", message)
+		case strings.Contains(errStr, "connection reset"):
+			message = fmt.Sprintf("%s: Connection was reset - network may be unstable, retrying...", message)
+		case strings.Contains(errStr, "TLS handshake timeout"):
+			message = fmt.Sprintf("%s: TLS handshake timeout - server may be busy, retrying...", message)
+		case strings.Contains(errStr, "too many open files"):
+			message = fmt.Sprintf("%s: System file limit reached - waiting for resources to be freed, retrying...", message)
+		case strings.Contains(errStr, "503"):
+			message = fmt.Sprintf("%s: Service temporarily unavailable (503) - server is overloaded, retrying...", message)
+		case strings.Contains(errStr, "502"):
+			message = fmt.Sprintf("%s: Bad gateway (502) - upstream server may be unavailable, retrying...", message)
+		case strings.Contains(errStr, "504"):
+			message = fmt.Sprintf("%s: Gateway timeout (504) - upstream server is not responding, retrying...", message)
+		case strings.Contains(errStr, "429"):
+			message = fmt.Sprintf("%s: Too many requests (429) - rate limit exceeded, retrying...", message)
+		default:
+			message = fmt.Sprintf("%s: %s", message, errStr)
+		}
 	}
 
 	if err != nil || runner.ErrorMessage != "" {