Fixing lint issues

Azure · Feb 9, 2024 · 8f90dfc · 8f90dfc
1 parent 3b2c4a6
commit 8f90dfc
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 4 deletions.
diff --git a/pkg/backend/openshiftcluster.go b/pkg/backend/openshiftcluster.go
@@ -347,7 +347,10 @@ func (ocb *openShiftClusterBackend) asyncOperationResultLog(log *logrus.Entry, i
 
 	err, ok := backendErr.(*api.CloudError)
 	if ok {
-		if int(err.StatusCode) < 500 {
+		if err.StatusCode < 300 && err.StatusCode >= 200 {
+			log.Info("long running operation succeeded")
+			return
+		} else if err.StatusCode < 500 {
 			log = log.WithField("resultType", utillog.UserErrorResultType)
 		} else {
 			log = log.WithField("resultType", utillog.ServerErrorResultType)

diff --git a/pkg/frontend/frontend.go b/pkg/frontend/frontend.go
@@ -475,15 +475,14 @@ func frontendOperationResultLog(log *logrus.Entry, method string, err error) {
 
 	switch err := err.(type) {
 	case *api.CloudError:
-		if int(err.StatusCode) < 300 && int(err.StatusCode) >= 200 {
+		if err.StatusCode < 300 && err.StatusCode >= 200 {
 			log.Info("front end operation succeeded")
 			return
-		} else if int(err.StatusCode) < 500 {
+		} else if err.StatusCode < 500 {
 			log = log.WithField("resultType", utillog.UserErrorResultType)
 		} else {
 			log = log.WithField("resultType", utillog.ServerErrorResultType)
 		}
-		log = log.WithField("resultType", utillog.UserErrorResultType)
 	case statusCodeError:
 		if int(err) < 300 && int(err) >= 200 {
 			log.Info("front end operation succeeded")