K8SPSMDB-754 update logs level (#1683)

Co-authored-by: Viacheslav Sarzhan <slava.sarzhan@percona.com>

Author: nmarukovich

pkg/apis/psmdb/v1/psmdb_types.go

@@ -410,7 +410,7 @@ func (m *MultiAZ) WithSidecarVolumes(log logr.Logger, volumes []corev1.Volume) [
 
 	for _, v := range m.SidecarVolumes {
 		if _, ok := names[v.Name]; ok {
-			log.Info("Wrong sidecar volume name, it is skipped", "volumeName", v.Name)
+			log.Error(errors.New("Wrong sidecar volume name, it is skipped"), "volumeName", v.Name)
 			continue
 		}
 
@@ -431,7 +431,7 @@ func (m *MultiAZ) WithSidecarPVCs(log logr.Logger, pvcs []corev1.PersistentVolum
 
 	for _, p := range m.SidecarPVCs {
 		if _, ok := names[p.Name]; ok {
-			log.Info("Wrong sidecar PVC name, it is skipped", "PVCName", p.Name)
+			log.Error(errors.New("Wrong sidecar PVC name, it is skipped"), "PVCName", p.Name)
 			continue
 		}
 

pkg/controller/perconaservermongodb/finalizers.go

@@ -120,7 +120,7 @@ func (r *ReconcilePerconaServerMongoDB) deletePSMDBPods(ctx context.Context, cr
 			rsDeleted = false
 			switch err {
 			case errWaitingTermination, errWaitingFirstPrimary:
-				log.Info(err.Error(), "rs", rs.Name)
+				log.Error(err, "rs", rs.Name)
 				continue
 			default:
 				log.Error(err, "failed to delete rs pods", "rs", rs.Name)

pkg/controller/perconaservermongodb/users.go

@@ -40,8 +40,8 @@ func (r *ReconcilePerconaServerMongoDB) reconcileUsers(ctx context.Context, cr *
 	}
 
 	if !cr.Spec.PMM.HasSecret(&sysUsersSecretObj) && cr.Spec.PMM.Enabled {
-		log.Info(fmt.Sprintf(`Can't enable PMM: "%s" or "%s" with "%s" keys don't exist in the secrets, or secrets and internal secrets are out of sync`,
-			api.PMMAPIKey, api.PMMUserKey, api.PMMPasswordKey), "secrets", cr.Spec.Secrets.Users, "internalSecrets", api.InternalUserSecretName(cr))
+		log.Error(errors.New(fmt.Sprintf(`Can't enable PMM: "%s" or "%s" with "%s" keys don't exist in the secrets, or secrets and internal secrets are out of sync`,
+			api.PMMAPIKey, api.PMMUserKey, api.PMMPasswordKey)), "secrets", cr.Spec.Secrets.Users, "internalSecrets", api.InternalUserSecretName(cr))
 	}
 
 	secretName := api.InternalUserSecretName(cr)

pkg/psmdb/tls/certmanager.go

@@ -240,7 +240,7 @@ func (c *certManagerController) Check(ctx context.Context, config *rest.Config,
 		case cmapichecker.ErrCertManagerCRDsNotFound:
 			return ErrCertManagerNotFound
 		case cmapichecker.ErrWebhookCertificateFailure, cmapichecker.ErrWebhookServiceFailure, cmapichecker.ErrWebhookDeploymentFailure:
-			log.Info("cert-manager is not ready", "error", cmapichecker.TranslateToSimpleError(err))
+			log.Error(cmapichecker.TranslateToSimpleError(err), "cert-manager is not ready")
 			return ErrCertManagerNotReady
 		}
 		return err