optimize account controller logger info

labring · May 14, 2024 · aee0a91 · aee0a91
1 parent 8cca809
commit aee0a91
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 6 deletions.
diff --git a/controllers/account/controllers/debt_controller.go b/controllers/account/controllers/debt_controller.go
@@ -156,14 +156,14 @@ func (r *DebtReconciler) reconcile(ctx context.Context, owner string) error {
 	// In a multi-region scenario, select the region where the account is created for SMS notification
 	smsEnable := account.CreateRegionID == r.LocalRegionID
 
-	r.Logger.Info("reconcile debt", "account", owner, "balance", account.Balance, "deduction balance", account.DeductionBalance)
+	//r.Logger.Info("reconcile debt", "account", owner, "balance", account.Balance, "deduction balance", account.DeductionBalance)
 	if err := r.Get(ctx, client.ObjectKey{Name: GetDebtName(owner), Namespace: r.accountSystemNamespace}, debt); client.IgnoreNotFound(err) != nil {
 		return err
 	} else if err != nil {
 		if err := r.syncDebt(ctx, owner, debt); err != nil {
 			return err
 		}
-		r.Logger.Info("create or update debt success", "debt", debt)
+		//r.Logger.Info("create or update debt success", "debt", debt)
 	}
 
 	nsList, err := getOwnNsList(r.Client, getUsername(owner))
@@ -326,7 +326,7 @@ func (r *DebtReconciler) reconcileDebtStatus(ctx context.Context, debt *accountv
 	}
 
 	if update {
-		r.Logger.Info("update debt status", "account", debt.Spec.UserName,
+		r.Logger.V(1).Info("update debt status", "account", debt.Spec.UserName,
 			"last status", lastStatus, "last update time", time.Unix(debt.Status.LastUpdateTimestamp, 0).Format(time.RFC3339),
 			"current status", debt.Status.AccountDebtStatus, "time", time.Now().UTC().Format(time.RFC3339))
 		return r.Status().Update(ctx, debt)

diff --git a/controllers/account/controllers/namespace_controller.go b/controllers/account/controllers/namespace_controller.go
@@ -99,16 +99,14 @@ func (r *NamespaceReconciler) Reconcile(ctx context.Context, req ctrl.Request) (
 		logger.Error(fmt.Errorf("no debt status"), "no debt status")
 		return ctrl.Result{}, nil
 	}
-	logger.Info("debt status", "status", debtStatus)
+	logger.V(1).Info("debt status", "status", debtStatus)
 	switch debtStatus {
 	case v1.SuspendDebtNamespaceAnnoStatus:
-		logger.Info("suspend namespace resources")
 		if err := r.SuspendUserResource(ctx, req.NamespacedName.Name); err != nil {
 			logger.Error(err, "suspend namespace resources failed")
 			return ctrl.Result{}, err
 		}
 	case v1.ResumeDebtNamespaceAnnoStatus:
-		logger.Info("resume namespace resources")
 		if err := r.ResumeUserResource(ctx, req.NamespacedName.Name); err != nil {
 			logger.Error(err, "resume namespace resources failed")
 			return ctrl.Result{}, err