diff --git a/controllers/deployment_controller.go b/controllers/deployment_controller.go index b4e0869..c00246c 100644 --- a/controllers/deployment_controller.go +++ b/controllers/deployment_controller.go @@ -74,7 +74,7 @@ type DeploymentReconciler struct { // - https://pkg.go.dev/sigs.k8s.io/controller-runtime/pkg/reconcile func (r *DeploymentReconciler) Reconcile(ctx context.Context, req ctrl.Request) (ctrl.Result, error) { reqLogger := logDeployment.WithValues("Request.Namespace", req.Namespace, "Request.Name", req.Name) - reqLogger.V(int(logs.DebugLevel)).Info("Reconciling Deployment") + reqLogger.V(logs.DebugLevel).Info("Reconciling Deployment") deployment := &appsv1.Deployment{} err := r.Get(context.Background(), req.NamespacedName, deployment) @@ -87,7 +87,7 @@ func (r *DeploymentReconciler) Reconcile(ctx context.Context, req ctrl.Request) annotations, annotationsFound := op.GetAnnotationsForDeployment(deployment, r.OpAnnotationRegExp) if !annotationsFound { - reqLogger.V(int(logs.DebugLevel)).Info("No 1Password Annotations found") + reqLogger.V(logs.DebugLevel).Info("No 1Password Annotations found") return ctrl.Result{}, nil } diff --git a/controllers/onepassworditem_controller.go b/controllers/onepassworditem_controller.go index 51afefa..f4a2831 100644 --- a/controllers/onepassworditem_controller.go +++ b/controllers/onepassworditem_controller.go @@ -79,7 +79,7 @@ type OnePasswordItemReconciler struct { // - https://pkg.go.dev/sigs.k8s.io/controller-runtime/pkg/reconcile func (r *OnePasswordItemReconciler) Reconcile(ctx context.Context, req ctrl.Request) (ctrl.Result, error) { reqLogger := logOnePasswordItem.WithValues("Request.Namespace", req.Namespace, "Request.Name", req.Name) - reqLogger.V(int(logs.DebugLevel)).Info("Reconciling OnePasswordItem") + reqLogger.V(logs.DebugLevel).Info("Reconciling OnePasswordItem") onepassworditem := &onepasswordv1.OnePasswordItem{} err := r.Get(context.Background(), req.NamespacedName, onepassworditem) diff --git a/pkg/logs/log_levels.go b/pkg/logs/log_levels.go index 9e72789..e2b20f3 100644 --- a/pkg/logs/log_levels.go +++ b/pkg/logs/log_levels.go @@ -3,11 +3,9 @@ package logs // A Level is a logging priority. Lower levels are more important. // All levels have been multipled by -1 to ensure compatibilty // between zapcore and logr -type Level int - const ( - ErrorLevel Level = iota - 2 - WarnLevel - InfoLevel - DebugLevel + ErrorLevel = -2 + WarnLevel = -1 + InfoLevel = 0 + DebugLevel = 1 ) diff --git a/pkg/onepassword/secret_update_handler.go b/pkg/onepassword/secret_update_handler.go index 3b65e9e..a24e7e1 100644 --- a/pkg/onepassword/secret_update_handler.go +++ b/pkg/onepassword/secret_update_handler.go @@ -83,7 +83,7 @@ func (h *SecretUpdateHandler) restartDeploymentsWithUpdatedSecrets(updatedSecret } } - log.V(int(logs.DebugLevel)).Info(fmt.Sprintf("Deployment %q at namespace %q is up to date", deployment.GetName(), deployment.Namespace)) + log.V(logs.DebugLevel).Info(fmt.Sprintf("Deployment %q at namespace %q is up to date", deployment.GetName(), deployment.Namespace)) } return nil @@ -132,7 +132,7 @@ func (h *SecretUpdateHandler) updateKubernetesSecrets() (map[string]map[string]* if currentVersion != itemVersion || secret.Annotations[ItemPathAnnotation] != itemPathString { if isItemLockedForForcedRestarts(item) { - log.V(int(logs.DebugLevel)).Info(fmt.Sprintf("Secret '%v' has been updated in 1Password but is set to be ignored. Updates to an ignored secret will not trigger an update to a kubernetes secret or a rolling restart.", secret.GetName())) + log.V(logs.DebugLevel).Info(fmt.Sprintf("Secret '%v' has been updated in 1Password but is set to be ignored. Updates to an ignored secret will not trigger an update to a kubernetes secret or a rolling restart.", secret.GetName())) secret.Annotations[VersionAnnotation] = itemVersion secret.Annotations[ItemPathAnnotation] = itemPathString if err := h.client.Update(context.Background(), &secret); err != nil { @@ -145,7 +145,7 @@ func (h *SecretUpdateHandler) updateKubernetesSecrets() (map[string]map[string]* secret.Annotations[VersionAnnotation] = itemVersion secret.Annotations[ItemPathAnnotation] = itemPathString secret.Data = kubeSecrets.BuildKubernetesSecretData(item.Fields, item.Files) - log.V(int(logs.DebugLevel)).Info(fmt.Sprintf("New secret path: %v and version: %v", secret.Annotations[ItemPathAnnotation], secret.Annotations[VersionAnnotation])) + log.V(logs.DebugLevel).Info(fmt.Sprintf("New secret path: %v and version: %v", secret.Annotations[ItemPathAnnotation], secret.Annotations[VersionAnnotation])) if err := h.client.Update(context.Background(), &secret); err != nil { log.Error(err, "failed to update secret %s to version %d: %s", secret.Name, itemVersion, err) continue diff --git a/pkg/utils/k8sutil.go b/pkg/utils/k8sutil.go index b2252ba..c4c6fdb 100644 --- a/pkg/utils/k8sutil.go +++ b/pkg/utils/k8sutil.go @@ -55,7 +55,7 @@ func GetOperatorNamespace() (string, error) { return "", err } ns := strings.TrimSpace(string(nsBytes)) - log.V(int(logs.DebugLevel)).Info("Found namespace", "Namespace", ns) + log.V(logs.DebugLevel).Info("Found namespace", "Namespace", ns) return ns, nil }