Merge pull request #598 from googs1025/fix_log

avoid log panic
kubernetes-sigs · Jun 7, 2024 · c270059 · c270059
2 parents 4093bad + 46a3902
commit c270059
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/pkg/controllers/pod_controller.go b/pkg/controllers/pod_controller.go
@@ -150,7 +150,7 @@ func (r *PodReconciler) Reconcile(ctx context.Context, req ctrl.Request) (ctrl.R
 		return ctrl.Result{}, err
 	}
 	if !valid {
-		log.V(2).Info("deleting follower pods for leader pod: %s", leaderPod.Name)
+		log.V(2).Info(fmt.Sprintf("deleting follower pods for leader pod: %s", leaderPod.Name))
 		return ctrl.Result{}, r.deleteFollowerPods(ctx, podList.Items)
 	}
 	return ctrl.Result{}, nil

diff --git a/pkg/webhooks/pod_admission_webhook.go b/pkg/webhooks/pod_admission_webhook.go
@@ -83,7 +83,7 @@ func (p *podWebhook) leaderPodScheduled(ctx context.Context, pod *corev1.Pod) (b
 	}
 	scheduled := leaderPod.Spec.NodeName != ""
 	if !scheduled {
-		log.V(2).Info("leader pod %s is not yet scheduled", leaderPod.Name)
+		log.V(2).Info(fmt.Sprintf("leader pod %s is not yet scheduled", leaderPod.Name))
 	}
 	return scheduled, nil
 }