refactor: Reduce log verbosity for skipped handlers (#692)

This should reduce noise in logs.

Author: jimmidyson

pkg/handlers/generic/lifecycle/ccm/handler.go

@@ -79,7 +79,7 @@ func (c *CCMHandler) AfterControlPlaneInitialized(
 	_, err := variables.Get[v1alpha1.CCM](varMap, c.variableName, c.variablePath...)
 	if err != nil {
 		if variables.IsNotFoundError(err) {
-			log.V(4).Info("Skipping CCM handler.")
+			log.V(5).Info("Skipping CCM handler.")
 			return
 		}
 		log.Error(

pkg/handlers/generic/lifecycle/clusterautoscaler/handler.go

@@ -94,7 +94,7 @@ func (n *DefaultClusterAutoscaler) AfterControlPlaneInitialized(
 		n.variablePath...)
 	if err != nil {
 		if variables.IsNotFoundError(err) {
-			log.Info(
+			log.V(5).Info(
 				"Skipping cluster-autoscaler handler, cluster does not specify request cluster-autoscaler addon deployment",
 			)
 			return

pkg/handlers/generic/lifecycle/cni/calico/handler.go

@@ -91,7 +91,7 @@ func (c *CalicoCNI) AfterControlPlaneInitialized(
 	cniVar, err := variables.Get[v1alpha1.CNI](varMap, c.variableName, c.variablePath...)
 	if err != nil {
 		if variables.IsNotFoundError(err) {
-			log.
+			log.V(5).
 				Info(
 					"Skipping Calico CNI handler, cluster does not specify request CNI addon deployment",
 				)
@@ -110,7 +110,7 @@ func (c *CalicoCNI) AfterControlPlaneInitialized(
 		return
 	}
 	if cniVar.Provider != v1alpha1.CNIProviderCalico {
-		log.Info(
+		log.V(5).Info(
 			fmt.Sprintf(
 				"Skipping Calico CNI handler, cluster does not specify %q as value of CNI provider variable",
 				v1alpha1.CNIProviderCalico,

pkg/handlers/generic/lifecycle/cni/calico/strategy_crs.go

@@ -65,7 +65,7 @@ func (s crsStrategy) apply(
 	infraKind := req.Cluster.Spec.InfrastructureRef.Kind
 	defaultInstallationConfigMapName, ok := s.config.defaultProviderInstallationConfigMapNames[infraKind]
 	if !ok {
-		log.V(4).Info(
+		log.Info(
 			fmt.Sprintf(
 				"Skipping Calico CNI handler, no default installation ConfigMap configured for infrastructure provider %q",
 				req.Cluster.Spec.InfrastructureRef.Kind,

pkg/handlers/generic/lifecycle/cni/cilium/handler.go

@@ -91,7 +91,7 @@ func (c *CiliumCNI) AfterControlPlaneInitialized(
 	cniVar, err := variables.Get[v1alpha1.CNI](varMap, c.variableName, c.variablePath...)
 	if err != nil {
 		if variables.IsNotFoundError(err) {
-			log.
+			log.V(5).
 				Info(
 					"Skipping Cilium CNI handler, cluster does not specify request CNI addon deployment",
 				)
@@ -110,7 +110,7 @@ func (c *CiliumCNI) AfterControlPlaneInitialized(
 		return
 	}
 	if cniVar.Provider != v1alpha1.CNIProviderCilium {
-		log.Info(
+		log.V(5).Info(
 			fmt.Sprintf(
 				"Skipping Cilium CNI handler, cluster does not specify %q as value of CNI provider variable",
 				v1alpha1.CNIProviderCilium,

pkg/handlers/generic/lifecycle/nfd/handler.go

@@ -91,7 +91,8 @@ func (n *DefaultNFD) AfterControlPlaneInitialized(
 	cniVar, err := variables.Get[v1alpha1.NFD](varMap, n.variableName, n.variablePath...)
 	if err != nil {
 		if variables.IsNotFoundError(err) {
-			log.Info("Skipping NFD handler, cluster does not specify request NFDaddon deployment")
+			log.V(5).
+				Info("Skipping NFD handler, cluster does not specify request NFDaddon deployment")
 			return
 		}
 		log.Error(

pkg/handlers/generic/lifecycle/serviceloadbalancer/handler.go

@@ -74,7 +74,7 @@ func (c *ServiceLoadBalancerHandler) AfterControlPlaneInitialized(
 		c.variablePath...)
 	if err != nil {
 		if variables.IsNotFoundError(err) {
-			log.
+			log.V(5).
 				Info(
 					"Skipping ServiceLoadBalancer, field is not specified",
 					"error",