From d5503c2ed2df272ec1bd94ebd223408fad29df1f Mon Sep 17 00:00:00 2001 From: joshvanl Date: Tue, 30 Nov 2021 15:13:14 +0000 Subject: [PATCH] Change certificates controller to no longer error for a Certificate that no longer exists Signed-off-by: joshvanl --- pkg/controller/certificates/issuing/issuing_controller.go | 2 +- pkg/controller/certificates/keymanager/keymanager_controller.go | 2 +- pkg/controller/certificates/readiness/readiness_controller.go | 2 +- .../certificates/requestmanager/requestmanager_controller.go | 2 +- .../certificates/revisionmanager/revisionmanager_controller.go | 2 +- pkg/controller/certificates/trigger/trigger_controller.go | 2 +- 6 files changed, 6 insertions(+), 6 deletions(-) diff --git a/pkg/controller/certificates/issuing/issuing_controller.go b/pkg/controller/certificates/issuing/issuing_controller.go index c14776da7..ab0fc05bf 100644 --- a/pkg/controller/certificates/issuing/issuing_controller.go +++ b/pkg/controller/certificates/issuing/issuing_controller.go @@ -149,7 +149,7 @@ func (c *controller) ProcessItem(ctx context.Context, key string) error { crt, err := c.certificateLister.Certificates(namespace).Get(name) if apierrors.IsNotFound(err) { - log.Error(err, "certificate not found for key") + log.V(logf.DebugLevel).Info("certificate not found for key", "error", err.Error()) return nil } if err != nil { diff --git a/pkg/controller/certificates/keymanager/keymanager_controller.go b/pkg/controller/certificates/keymanager/keymanager_controller.go index 2bade4fc4..1f36c1ff3 100644 --- a/pkg/controller/certificates/keymanager/keymanager_controller.go +++ b/pkg/controller/certificates/keymanager/keymanager_controller.go @@ -136,7 +136,7 @@ func (c *controller) ProcessItem(ctx context.Context, key string) error { crt, err := c.certificateLister.Certificates(namespace).Get(name) if apierrors.IsNotFound(err) { - log.Error(err, "certificate not found for key") + log.V(logf.DebugLevel).Info("certificate not found for key", "error", err.Error()) return nil } if err != nil { diff --git a/pkg/controller/certificates/readiness/readiness_controller.go b/pkg/controller/certificates/readiness/readiness_controller.go index 71ea651be..0f40be5f4 100644 --- a/pkg/controller/certificates/readiness/readiness_controller.go +++ b/pkg/controller/certificates/readiness/readiness_controller.go @@ -139,7 +139,7 @@ func (c *controller) ProcessItem(ctx context.Context, key string) error { crt, err := c.certificateLister.Certificates(namespace).Get(name) if apierrors.IsNotFound(err) { - log.Error(err, "certificate not found for key") + log.V(logf.DebugLevel).Info("certificate not found for key", "error", err.Error()) return nil } if err != nil { diff --git a/pkg/controller/certificates/requestmanager/requestmanager_controller.go b/pkg/controller/certificates/requestmanager/requestmanager_controller.go index 68858fc14..9a11c1151 100644 --- a/pkg/controller/certificates/requestmanager/requestmanager_controller.go +++ b/pkg/controller/certificates/requestmanager/requestmanager_controller.go @@ -133,7 +133,7 @@ func (c *controller) ProcessItem(ctx context.Context, key string) error { crt, err := c.certificateLister.Certificates(namespace).Get(name) if apierrors.IsNotFound(err) { - log.Error(err, "certificate not found for key") + log.V(logf.DebugLevel).Info("certificate not found for key", "error", err.Error()) return nil } if err != nil { diff --git a/pkg/controller/certificates/revisionmanager/revisionmanager_controller.go b/pkg/controller/certificates/revisionmanager/revisionmanager_controller.go index 166a82a5b..d6f0d16f2 100644 --- a/pkg/controller/certificates/revisionmanager/revisionmanager_controller.go +++ b/pkg/controller/certificates/revisionmanager/revisionmanager_controller.go @@ -103,7 +103,7 @@ func (c *controller) ProcessItem(ctx context.Context, key string) error { crt, err := c.certificateLister.Certificates(namespace).Get(name) if apierrors.IsNotFound(err) { - log.Error(err, "certificate not found for key") + log.V(logf.DebugLevel).Info("certificate not found for key", "error", err.Error()) return nil } if err != nil { diff --git a/pkg/controller/certificates/trigger/trigger_controller.go b/pkg/controller/certificates/trigger/trigger_controller.go index 4f146e54e..d25177624 100644 --- a/pkg/controller/certificates/trigger/trigger_controller.go +++ b/pkg/controller/certificates/trigger/trigger_controller.go @@ -134,7 +134,7 @@ func (c *controller) ProcessItem(ctx context.Context, key string) error { crt, err := c.certificateLister.Certificates(namespace).Get(name) if apierrors.IsNotFound(err) { - log.Error(err, "certificate not found for key") + log.V(logf.DebugLevel).Info("certificate not found for key", "error", err.Error()) return nil } if err != nil {