From fab777cdd580c81a319f2e1c1e08a46722b7c9f0 Mon Sep 17 00:00:00 2001 From: lengrongfu Date: Thu, 7 Jan 2021 12:34:01 +0800 Subject: [PATCH] add logging for policy creation and deletion events (#1445) * add logging for policy creation and deletion events * update log message * update log message kind type Co-authored-by: lengrongfu --- .gitignore | 2 +- pkg/policy/validate_controller.go | 12 ++++++++++-- 2 files changed, 11 insertions(+), 3 deletions(-) diff --git a/.gitignore b/.gitignore index d8cbb60872..03d0c8249e 100644 --- a/.gitignore +++ b/.gitignore @@ -8,4 +8,4 @@ coverage.txt cmd/initContainer/kyvernopre cmd/kyverno/kyverno cmd/cli/kubectl-kyverno/kyverno -kubectl-kyverno +kubectl-kyverno \ No newline at end of file diff --git a/pkg/policy/validate_controller.go b/pkg/policy/validate_controller.go index 98a3988ce6..8620ae7232 100644 --- a/pkg/policy/validate_controller.go +++ b/pkg/policy/validate_controller.go @@ -181,6 +181,9 @@ func (pc *PolicyController) canBackgroundProcess(p *kyverno.ClusterPolicy) bool func (pc *PolicyController) addPolicy(obj interface{}) { logger := pc.log p := obj.(*kyverno.ClusterPolicy) + + logger.Info("policy created event", "uid", p.UID, "kind", "ClusterPolicy", "policy_name", p.Name) + if !pc.canBackgroundProcess(p) { return } @@ -225,7 +228,7 @@ func (pc *PolicyController) deletePolicy(obj interface{}) { } } - logger.V(4).Info("deleting policy", "name", p.Name) + logger.Info("policy deleted event", "uid", p.UID, "kind", "ClusterPolicy", "policy_name", p.Name) // we process policies that are not set of background processing // as we need to clean up GRs when a policy is deleted @@ -236,6 +239,9 @@ func (pc *PolicyController) deletePolicy(obj interface{}) { func (pc *PolicyController) addNsPolicy(obj interface{}) { logger := pc.log p := obj.(*kyverno.Policy) + + logger.Info("policy created event", "uid", p.UID, "kind", "Policy", "policy_name", p.Name, "namespaces", p.Namespace) + pol := ConvertPolicyToClusterPolicy(p) if !pc.canBackgroundProcess(pol) { return @@ -279,8 +285,10 @@ func (pc *PolicyController) deleteNsPolicy(obj interface{}) { return } } + + logger.Info("policy deleted event", "uid", p.UID, "kind", "Policy", "policy_name", p.Name, "namespaces", p.Namespace) + pol := ConvertPolicyToClusterPolicy(p) - logger.V(4).Info("deleting namespace policy", "namespace", pol.Namespace, "name", pol.Name) // we process policies that are not set of background processing // as we need to clean up GRs when a policy is deleted