refactor(output): replace log.Info and log.Infof statements with Printf and log.Debugf statements (goharbor#884)

Author: qcserestipy

cmd/harbor/root/context/delete.go

@@ -81,9 +81,9 @@ If you specify --name, that credential (rather than the "current" one) will be u
 				}
 
 				if found {
-					logrus.Infof("Removed credential '%s' and cleared CurrentCredentialName", currentName)
+					logrus.Debugf("Removed credential '%s' and cleared CurrentCredentialName", currentName)
 				} else {
-					logrus.Infof("No credential named '%s' found; cleared CurrentCredentialName anyway", currentName)
+					logrus.Debugf("No credential named '%s' found; cleared CurrentCredentialName anyway", currentName)
 				}
 
 				return nil

cmd/harbor/root/context/update.go

@@ -69,8 +69,7 @@ If you specify --name, that credential (rather than the "current" one) will be u
 
 			// 5. Confirm to the user (logrus.Info is fine here; no error)
 			canonicalPath := strings.Join(actualSegments, ".")
-			logrus.Infof("Successfully updated %s to '%s'", canonicalPath, newValue)
-
+			fmt.Printf("Successfully updated %s to '%s'\n", canonicalPath, newValue)
 			return nil
 		},
 	}

cmd/harbor/root/project/list.go

@@ -93,7 +93,7 @@ func ListProjectCommand() *cobra.Command {
 
 			log.WithField("count", len(allProjects)).Debug("Number of projects fetched")
 			if len(allProjects) == 0 {
-				log.Info("No projects found")
+				fmt.Println("No projects found")
 				return nil
 			}
 			formatFlag := viper.GetString("output-format")

cmd/harbor/root/project/robot/create.go

@@ -102,7 +102,7 @@ Examples:
 				if loadErr != nil {
 					return fmt.Errorf("failed to load robot config from file: %v", loadErr)
 				}
-				logrus.Info("Successfully loaded robot configuration")
+				logrus.Debug("Successfully loaded robot configuration")
 				opts = *loadedOpts
 				if opts.ProjectName == "" {
 					opts.ProjectName = opts.Permissions[0].Namespace
@@ -196,7 +196,7 @@ Examples:
 				return fmt.Errorf("failed to create robot: %v", utils.ParseHarborErrorMsg(err))
 			}
 
-			logrus.Infof("Successfully created robot account '%s' (ID: %d)",
+			fmt.Printf("Successfully created robot account '%s' (ID: %d)\n",
 				response.Payload.Name, response.Payload.ID)
 
 			FormatFlag := viper.GetString("output-format")
@@ -209,7 +209,7 @@ Examples:
 			name, secret := response.Payload.Name, response.Payload.Secret
 
 			if exportToFile {
-				logrus.Info("Exporting robot credentials to file")
+				fmt.Printf("Exporting robot credentials to file\n")
 				exportSecretToFile(name, secret, response.Payload.CreationTime.String(), response.Payload.ExpiresAt)
 				return nil
 			} else {

cmd/harbor/root/project/robot/refresh.go

@@ -106,7 +106,7 @@ Examples:
 				return fmt.Errorf("failed to refresh robot secret: %v\n", err)
 			}
 
-			log.Info("Secret updated successfully.")
+			fmt.Println("Secret updated successfully.")
 
 			if response.Payload.Secret != "" {
 				secret = response.Payload.Secret

cmd/harbor/root/quota/update.go

@@ -22,7 +22,6 @@ import (
 	"github.com/goharbor/harbor-cli/pkg/prompt"
 	"github.com/goharbor/harbor-cli/pkg/utils"
 	"github.com/goharbor/harbor-cli/pkg/views/quota/update"
-	log "github.com/sirupsen/logrus"
 	"github.com/spf13/cobra"
 )
 
@@ -80,7 +79,7 @@ func UpdateQuotaCommand() *cobra.Command {
 				return fmt.Errorf("failed to update quota: %v", err)
 			}
 
-			log.Infof("quota updated successfully!")
+			fmt.Println("Quota updated successfully!")
 
 			return nil
 		},

cmd/harbor/root/registry/list.go

@@ -49,7 +49,7 @@ func ListRegistryCommand() *cobra.Command {
 				return fmt.Errorf("failed to get projects list: %v", err)
 			}
 			if len(registry.Payload) == 0 {
-				log.Info("No registries found")
+				fmt.Println("No registries found")
 				return nil
 			}
 			formatFlag := viper.GetString("output-format")

cmd/harbor/root/replication/executions/list.go

@@ -68,7 +68,7 @@ func ListCommand() *cobra.Command {
 
 			log.WithField("count", len(executions.Payload)).Debug("Number of executions fetched")
 			if len(executions.Payload) == 0 {
-				log.Info("No executions found")
+				fmt.Println("No executions found")
 				return nil
 			}
 

cmd/harbor/root/replication/policies/list.go

@@ -52,7 +52,7 @@ func ListCommand() *cobra.Command {
 
 			log.WithField("count", len(allPolicies.Payload)).Debug("Number of policies fetched")
 			if len(allPolicies.Payload) == 0 {
-				log.Info("No policies found")
+				fmt.Println("No policies found")
 				return nil
 			}
 

cmd/harbor/root/replication/policies/update.go

@@ -96,7 +96,7 @@ func UpdateCommand() *cobra.Command {
 				}
 			}
 
-			log.Infof("Updating replication policy: %s (ID: %d)", existingPolicy.Payload.Name, policyID)
+			log.Debugf("Updating replication policy: %s (ID: %d)", existingPolicy.Payload.Name, policyID)
 			create.CreateRPolicyView(createView, true)
 
 			var updatedPolicy *models.ReplicationPolicy
@@ -114,7 +114,7 @@ func UpdateCommand() *cobra.Command {
 				return fmt.Errorf("failed to update replication policy: %w", err)
 			}
 
-			log.Infof("Successfully updated replication policy: %s (ID: %d)", updatedPolicy.Name, policyID)
+			fmt.Printf("Successfully updated replication policy: %s (ID: %d)\n", updatedPolicy.Name, policyID)
 			return nil
 		},
 	}