From 7cdb0f627dadab9ae9af791e64690e49b7ee5533 Mon Sep 17 00:00:00 2001 From: Darren Lau Date: Mon, 7 Feb 2022 09:46:42 -0500 Subject: [PATCH] Remove duplicated info by using WithValues() --- controllers/kafkacluster_controller.go | 2 +- controllers/kafkatopic_controller.go | 2 +- controllers/kafkauser_controller.go | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a/controllers/kafkacluster_controller.go b/controllers/kafkacluster_controller.go index 399e08210..4bf5b3405 100644 --- a/controllers/kafkacluster_controller.go +++ b/controllers/kafkacluster_controller.go @@ -84,7 +84,7 @@ type KafkaClusterReconciler struct { // +kubebuilder:rbac:groups=networking.istio.io,resources=*,verbs=* func (r *KafkaClusterReconciler) Reconcile(ctx context.Context, request ctrl.Request) (ctrl.Result, error) { - log := logr.FromContextOrDiscard(ctx).WithValues("Request.Namespace", request.NamespacedName, "Request.Name", request.Name) + log := logr.FromContextOrDiscard(ctx) log.Info("Reconciling KafkaCluster") diff --git a/controllers/kafkatopic_controller.go b/controllers/kafkatopic_controller.go index 3e5266077..928c37a88 100644 --- a/controllers/kafkatopic_controller.go +++ b/controllers/kafkatopic_controller.go @@ -99,7 +99,7 @@ type KafkaTopicReconciler struct { // Reconcile reconciles the kafka topic func (r *KafkaTopicReconciler) Reconcile(ctx context.Context, request reconcile.Request) (reconcile.Result, error) { - reqLogger := logr.FromContextOrDiscard(ctx).WithValues("kafkatopic", request.NamespacedName, "Request.Name", request.Name) + reqLogger := logr.FromContextOrDiscard(ctx) reqLogger.Info("Reconciling KafkaTopic") var err error diff --git a/controllers/kafkauser_controller.go b/controllers/kafkauser_controller.go index 1e7bbc28f..e41516bcc 100644 --- a/controllers/kafkauser_controller.go +++ b/controllers/kafkauser_controller.go @@ -167,7 +167,7 @@ type KafkaUserReconciler struct { // Reconcile reads that state of the cluster for a KafkaUser object and makes changes based on the state read // and what is in the KafkaUser.Spec func (r *KafkaUserReconciler) Reconcile(ctx context.Context, request reconcile.Request) (reconcile.Result, error) { - reqLogger := logr.FromContextOrDiscard(ctx).WithValues("kafkauser", request.NamespacedName, "Request.Name", request.Name) + reqLogger := logr.FromContextOrDiscard(ctx) reqLogger.Info("Reconciling KafkaUser") var err error