fix(controller): change fmt error format

topfreegames · Sep 8, 2020 · fd366f7 · fd366f7
1 parent 3fcff20
commit fd366f7
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/controller/controller.go b/controller/controller.go
@@ -392,7 +392,7 @@ func ScaleUp(
 
 	shouldNotProceedErr, err := shouldScaleUpProceed(l, redisClient, scheduler.Name, mr)
 	if err != nil {
-		return fmt.Errorf("failed to check whether scale up should proceed: %w", err)
+		return fmt.Errorf("failed to check whether scale up should proceed: %s", err)
 	}
 
 	if shouldNotProceedErr != nil {

diff --git a/controller/utils.go b/controller/utils.go
@@ -849,7 +849,7 @@ func shouldScaleUpProceed(
 	}
 
 	if err := podsInInvalidWaitingState(logger, redisClient, namespace, pods); err != nil {
-		return fmt.Errorf("invalid pod waiting state: %w", err), nil
+		return fmt.Errorf("invalid pod waiting state: %s", err), nil
 	}
 
 	// Everything is fine, scale up can proceed.

diff --git a/models/pod.go b/models/pod.go
@@ -430,7 +430,7 @@ func ValidatePodWaitingState(pod *Pod) error {
 	for _, invalidState := range InvalidPodWaitingStates {
 		status := &pod.Status
 		if checkWaitingReason(status, invalidState) {
-			return fmt.Errorf("%w: one or more containers in pod are in %s", InvalidPodWaitingStateErr, invalidState)
+			return fmt.Errorf("%s: one or more containers in pod are in %s", InvalidPodWaitingStateErr, invalidState)
 		}
 	}
 	return nil