From 0b66321902fd5307e0a826a4a9c0e27571f66ab1 Mon Sep 17 00:00:00 2001
From: Margo Crawford <margaretc@vmware.com>
Date: Wed, 19 May 2021 11:05:35 -0700
Subject: [PATCH] Changes to make the linter pass

---
 .../impersonatorconfig/impersonator_config.go          | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/internal/controller/impersonatorconfig/impersonator_config.go b/internal/controller/impersonatorconfig/impersonator_config.go
index cf3939fb..03ed6d24 100644
--- a/internal/controller/impersonatorconfig/impersonator_config.go
+++ b/internal/controller/impersonatorconfig/impersonator_config.go
@@ -462,7 +462,7 @@ func (c *impersonatorConfigController) ensureLoadBalancerIsStarted(ctx context.C
 				"service", c.generatedLoadBalancerServiceName,
 				"namespace", c.namespace)
 			_, err = c.k8sClient.CoreV1().Services(c.namespace).Update(ctx, &loadBalancer, metav1.UpdateOptions{})
-			// TODO: make sure this error is handled if necessary
+			return err
 		}
 		return nil
 	}
@@ -689,10 +689,10 @@ func (c *impersonatorConfigController) findDesiredTLSCertificateName(config *v1a
 	// - clusterip and no external endpoint
 	if config.ExternalEndpoint != "" {
 		return c.findTLSCertificateNameFromEndpointConfig(config), nil
-	} else if config.Service.Type == v1alpha1.ImpersonationProxyServiceTypeClusterIP {
-		// c.findTLSCertificateNameFromClusterIPService()
-		// TODO implement this
-	}
+	} // else if config.Service.Type == v1alpha1.ImpersonationProxyServiceTypeClusterIP {
+	//	// c.findTLSCertificateNameFromClusterIPService()
+	//	// TODO implement this
+	//}
 	return c.findTLSCertificateNameFromLoadBalancer()
 }