Only fetch service CA if we actually need to update a ConfigMap

Author: simu

controllers/configmap_controller.go

@@ -71,11 +71,6 @@ func (r *ConfigMapReconciler) Reconcile(ctx context.Context, req ctrl.Request) (
 		return ctrl.Result{}, err
 	}
 
-	serviceCA, err := certs.GetServiceCA(ctx, r.Client, l, r.CANamespace)
-	if err != nil {
-		l.Info("Service CA not ready yet, requeuing request")
-		return ctrl.Result{}, err
-	}
 	cmLabels := cm.Labels
 	if cmLabels == nil {
 		// nothing to do, exit
@@ -99,6 +94,12 @@ func (r *ConfigMapReconciler) Reconcile(ctx context.Context, req ctrl.Request) (
 		return ctrl.Result{}, nil
 	}
 
+	serviceCA, err := certs.GetServiceCA(ctx, r.Client, l, r.CANamespace)
+	if err != nil {
+		l.Info("Service CA not ready yet, requeuing request")
+		return ctrl.Result{}, err
+	}
+
 	origCM := cm.DeepCopy()
 	if cm.Data == nil {
 		cm.Data = map[string]string{}