build: Make golangci-lint logrlint happy.

Fix "odd number of arguments passed as key-value pairs for logging". Pass to logr key-value pairs for structured logging. Signed-off-by: Víctor Cuadrado Juan <vcuadradojuan@suse.de>
kubewarden · Oct 11, 2022 · 08ce746 · 08ce746
1 parent ccfe9ce
commit 08ce746
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/controllers/policyserver_controller.go b/controllers/policyserver_controller.go
@@ -125,7 +125,7 @@ func (r *PolicyServerReconciler) SetupWithManager(mgr ctrl.Manager) error {
 	err := mgr.GetFieldIndexer().IndexField(context.Background(), &policiesv1.ClusterAdmissionPolicy{}, constants.PolicyServerIndexKey, func(object client.Object) []string {
 		policy, ok := object.(*policiesv1.ClusterAdmissionPolicy)
 		if !ok {
-			r.Log.Error(nil, "object is not type of ClusterAdmissionPolicy: %#v", policy)
+			r.Log.Error(nil, "object is not type of ClusterAdmissionPolicy: %#v", "policy", policy)
 			return []string{}
 		}
 		return []string{policy.Spec.PolicyServer}
@@ -136,7 +136,7 @@ func (r *PolicyServerReconciler) SetupWithManager(mgr ctrl.Manager) error {
 	err = mgr.GetFieldIndexer().IndexField(context.Background(), &policiesv1.AdmissionPolicy{}, constants.PolicyServerIndexKey, func(object client.Object) []string {
 		policy, ok := object.(*policiesv1.AdmissionPolicy)
 		if !ok {
-			r.Log.Error(nil, "object is not type of ClusterAdmissionPolicy: %#v", policy)
+			r.Log.Error(nil, "object is not type of AdmissionPolicy: %#v", "policy", policy)
 			return []string{}
 		}
 		return []string{policy.Spec.PolicyServer}
@@ -153,7 +153,7 @@ func (r *PolicyServerReconciler) SetupWithManager(mgr ctrl.Manager) error {
 			// object.
 			policy, ok := object.(*policiesv1.AdmissionPolicy)
 			if !ok {
-				r.Log.Info("object is not type of AdmissionPolicy: %+v", policy)
+				r.Log.Info("object is not type of AdmissionPolicy: %+v", "policy", policy)
 				return []ctrl.Request{}
 			}
 
@@ -172,7 +172,7 @@ func (r *PolicyServerReconciler) SetupWithManager(mgr ctrl.Manager) error {
 			// object.
 			policy, ok := object.(*policiesv1.ClusterAdmissionPolicy)
 			if !ok {
-				r.Log.Info("object is not type of ClusterAdmissionPolicy: %+v", policy)
+				r.Log.Info("object is not type of ClusterAdmissionPolicy: %+v", "policy", policy)
 				return []ctrl.Request{}
 			}