refactor: Add more debug logs, remove redundant keys/values from log messages in preflight framework (#1246)

**What problem does this PR solve?**:
Adds more debug logs, and removes redundant keys/values from log
messages in the preflight framework.

**Which issue(s) this PR fixes**:
Fixes #

**How Has This Been Tested?**:
<!--
Please describe the tests that you ran to verify your changes.
Provide output from the tests and any manual steps needed to replicate
the tests.
-->

**Special notes for your reviewer**:
<!--
Use this to provide any additional information to the reviewers.
This may include:
- Best way to review the PR.
- Where the author wants the most review attention on.
- etc.
-->

Author: dlipovetsky

pkg/webhook/preflight/preflight.go

@@ -111,11 +111,15 @@ type namedResult struct {
 }
 
 func (h *WebhookHandler) Handle(ctx context.Context, req admission.Request) admission.Response {
+	log := ctrl.LoggerFrom(ctx)
+
 	if req.Operation == admissionv1.Delete {
+		log.V(5).Info("Skipping preflight checks for delete operation")
 		return admission.Allowed("")
 	}
 
 	if req.Operation == admissionv1.Update {
+		log.V(5).Info("Skipping preflight checks for update operation")
 		return admission.Allowed("")
 	}
 
@@ -139,6 +143,7 @@ func (h *WebhookHandler) Handle(ctx context.Context, req admission.Request) admi
 	skipEvaluator := skip.New(cluster)
 
 	if skipEvaluator.ForAll() {
+		log.V(5).Info("Skipping all preflight checks")
 		// If the cluster has skipped all checks, return allowed.
 		return admission.Allowed("").WithWarnings(
 			"Cluster has skipped all preflight checks",
@@ -149,6 +154,7 @@ func (h *WebhookHandler) Handle(ctx context.Context, req admission.Request) admi
 	// that we have time to summarize the results, and return a response.
 	checkTimeout := Timeout - 2*time.Second
 	checkCtx, checkCtxCancel := context.WithTimeout(ctx, checkTimeout)
+	log.V(5).Info("Running preflight checks")
 	resultsOrderedByCheckerAndCheck := run(checkCtx, h.client, cluster, skipEvaluator, h.checkers)
 	checkCtxCancel()
 
@@ -230,8 +236,6 @@ func run(ctx context.Context,
 				ctrl.LoggerFrom(ctx).V(5).Info(
 					"running preflight check",
 					"checkName", check.Name(),
-					"clusterName", cluster.Name,
-					"clusterNamespace", cluster.Namespace,
 				)
 				if skipEvaluator.For(check.Name()) {
 					resultsOrderedByCheck[j] = namedResult{
@@ -275,8 +279,6 @@ func run(ctx context.Context,
 								fmt.Errorf("preflight check panic"),
 								fmt.Sprintf("%v", r),
 								"checkName", check.Name(),
-								"clusterName", cluster.Name,
-								"clusterNamespace", cluster.Namespace,
 								"stackTrace", string(debug.Stack()),
 							)
 						}