fixup: log messages

Signed-off-by: KevFan <[email protected]>
Kuadrant · Oct 15, 2024 · 24cff44 · 24cff44
1 parent e95be4a
commit 24cff44
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 4 deletions.
diff --git a/controllers/effective_tls_policies_reconciler.go b/controllers/effective_tls_policies_reconciler.go
@@ -178,8 +178,6 @@ func (t *EffectiveTLSPoliciesReconciler) Reconcile(ctx context.Context, _ []cont
 }
 
 func (t *EffectiveTLSPoliciesReconciler) deleteCertificatesForPolicy(ctx context.Context, topology *machinery.Topology, p *kuadrantv1alpha1.TLSPolicy) error {
-	logger := controller.LoggerFromContext(ctx).WithName("EffectiveTLSPoliciesReconciler").WithName("deleteCertificatesForPolicy")
-
 	certs := lo.FilterMap(topology.Objects().Items(), func(item machinery.Object, index int) (*certmanv1.Certificate, bool) {
 		r, ok := item.(*controller.RuntimeObject)
 		if !ok {
@@ -198,7 +196,6 @@ func (t *EffectiveTLSPoliciesReconciler) deleteCertificatesForPolicy(ctx context
 		resource := t.client.Resource(CertManagerCertificatesResource).Namespace(cert.GetNamespace())
 
 		if err := resource.Delete(ctx, cert.Name, metav1.DeleteOptions{}); err != nil {
-			logger.Error(err, "delete certificate", "name", cert.Name)
 			return err
 		}
 	}

diff --git a/controllers/tlspolicy_status_updater.go b/controllers/tlspolicy_status_updater.go
@@ -47,7 +47,7 @@ func (t *TLSPolicyStatusUpdaterReconciler) Subscription() *controller.Subscripti
 }
 
 func (t *TLSPolicyStatusUpdaterReconciler) UpdateStatus(ctx context.Context, _ []controller.ResourceEvent, topology *machinery.Topology, _ error, s *sync.Map) error {
-	logger := controller.LoggerFromContext(ctx).WithName("TLSPolicyStatusUpdaterReconciler").WithName("Reconcile")
+	logger := controller.LoggerFromContext(ctx).WithName("TLSPolicyStatusUpdaterReconciler").WithName("UpdateStatus")
 
 	policies := lo.FilterMap(topology.Policies().Items(), func(item machinery.Policy, index int) (*kuadrantv1alpha1.TLSPolicy, bool) {
 		p, ok := item.(*kuadrantv1alpha1.TLSPolicy)